Repository navigation
fix: preserve app-defined HTTP statuses in responses - #788
Open
kossnocorp wants to merge 6 commits into
Open
kossnocorp wants to merge 6 commits into
kossnocorp wants to merge 6 commits into
Conversation
🚀 Deploying Preview to Cloudflare 🚀Preview URL: https://fix-784-error-codes-void-cloud.voidzero-docs.workers.dev (commit 0b9b5de)This URL reflects your latest Preview deploymentPreview Deployments by commit
|
kossnocorp
marked this pull request as ready for review
November 13, 2026 10:01
Member
|
@codex review |
Codex Review Summary: ✅ CompletedThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
Codex Review: Didn't find any major issues. Delightful! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
kossnocorp
force-pushed
the
fix/784-error-codes
branch
from
November 13, 2026 10:34
e9c4d68 to
e49f0f4
Compare
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The PR closes #784.
The changes:
Preserve app-defined HTTP statuses when rendering pages
Previously, Void could replace the loader's status with
200despitec.status(503):The page now renders with HTTP 503 for both document loads and client navigation. This also works when middleware sets the status, with validated loaders, and when the page streams deferred props.
Render expected loader failures through page error boundaries
An example of the app checking the user account state:
Previously, these exceptions returned plain text pages even when the app defined an
_errorcomponent. Client navigation fell back to a document load, replacing the application and its layouts.Now, response-less
HTTPExceptionrender through the nearest page boundary, preserving their status and public message. For example,pages/account/_errorcan display an account-specific error within the surrounding layouts._not-foundbehavior.resretain their explicit response behavior.Render application-middleware HTTP failures through the root boundary
Previously, this returned plain text and bypassed the app's root
_errorcomponent.Now, page
GET/HEADrequests render the root error boundary withHTTP 503and the supplied message, without running the page loader. Client navigation preserves the root layout instead of replacing the document.Exceptions from handler middleware use the nearest page boundary, like loader exceptions.