Skip to content

Throwaway PR: Try to repro and debug CI-specific test failures from #1539#1627

Closed
brandonpayton wants to merge 11 commits into
trunkfrom
fix-request-routing-test-failures
Closed

Throwaway PR: Try to repro and debug CI-specific test failures from #1539#1627
brandonpayton wants to merge 11 commits into
trunkfrom
fix-request-routing-test-failures

Conversation

@brandonpayton
Copy link
Copy Markdown
Member

This is a separate PR to allow experimentation and debugging of unexpected test failures under #1539 that cannot be reproduced locally.

@brandonpayton brandonpayton self-assigned this Jul 18, 2024
This PR:
- Updates PHPRequestHandler to support custom file-not-found responses
- Updates worker-thread to configure PHPRequestHandler to handle
   file-not-found conditions properly for WordPress
- Updates worker-thread to flag remote WP assets for retrieval by the
    Service Worker

Prior to this PR, we could not easily tell whether we should request a
missing static asset from the web or delegate a request for a missing
file to WordPress.

Related to #1365 where we need better information to judge whether to
handle a request for a missing static file as PHP.

- CI tests
- Test manually with `npm run dev` and observe that Playground loads
normally with no unexpected errors in the console
@brandonpayton brandonpayton force-pushed the fix-request-routing-test-failures branch from 3d886a5 to ad90d12 Compare July 18, 2024 21:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

No open projects
Archived in project

Development

Successfully merging this pull request may close these issues.

1 participant