Skip to content

fix(loader): respect path prefix for 404 head check #39348

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
serhalp wants to merge 2 commits into
base: master
Choose a base branch
from
Draft

fix(loader): respect path prefix for 404 head check #39348

serhalp wants to merge 2 commits into from
+45 −4

Conversation

@serhalp
Copy link
Member

@serhalp serhalp commented Aug 8, 2025

Summary

  • ensure 404 HTML HEAD requests include path prefix
  • add regression test for pathPrefix 404 behaviour

Testing

  • npx eslint packages/gatsby/cache-dir/loader.js packages/gatsby/cache-dir/__tests__/loader.js
  • yarn jest packages/gatsby/cache-dir/__tests__/loader.js

https://chatgpt.com/codex/tasks/task_e_689279740c9c8321bf1c14ea64e664b9
@gatsbot gatsbot bot added the status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer label Aug 8, 2025
@serhalp serhalp linked an issue Aug 8, 2025 that may be closed by this pull request
Page 404 not rendering react when using pathPrefix and redirect #32142
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

codex status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer

3 participants

@serhalp @pieh