escape Location header with JSON.stringify#2132
Conversation
✅ Deploy Preview for solid-start-landing-page ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
🦋 Changeset detectedLatest commit: 4f8000e The changes in this PR will be included in the next version bump. Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
commit: |
There was a problem hiding this comment.
Pull request overview
This PR updates the streamed “complete redirect” handling to escape the Location header value when embedding it into an inline <script> redirect, aiming to reduce script injection risk during SSR streaming redirects.
Changes:
- Serialize the
Locationheader value usingJSON.stringifybefore writing it into the inline redirect script.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
808177e to
d4fd44f
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
No description provided.