Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@pi0
Copy link
Member

@pi0 pi0 commented Oct 21, 2025

caught from nuxt ci

When preparing an error response (that is, returning a Response) itself, we should avoid double merging headers as the first handler itself merges them.


Another issue I found is that we are appending prepared headers (intended for a 200 OK response) to the final error response. It seems not best and might leak unwanted headers to error page. We might change that at least for built-in JSON error renderer => #1228

@pi0 pi0 marked this pull request as ready for review October 23, 2025 13:04
@pi0 pi0 merged commit 5191132 into main Oct 23, 2025
4 checks passed
@pi0 pi0 deleted the fix/nested-double-headers branch October 23, 2025 13:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant