Skip to content

[GHSA-8v8x-cx79-35w7] React Router SSR XSS in ScrollRestoration - #9371

Merged
advisory-database[bot] merged 1 commit into
HF-CYGG/advisory-improvement-9371from
HF-CYGG-GHSA-8v8x-cx79-35w7
Sep 8, 2026
Merged

[GHSA-8v8x-cx79-35w7] React Router SSR XSS in ScrollRestoration#9371
advisory-database[bot] merged 1 commit into
HF-CYGG/advisory-improvement-9371from
HF-CYGG-GHSA-8v8x-cx79-35w7

Conversation

@HF-CYGG

@HF-CYGG HF-CYGG commented Sep 5, 2026

Copy link
Copy Markdown

Updates

  • References

Comments
Add the missing upstream React Router v7 fix commit for GHSA-8v8x-cx79-35w7.

Commit c89c32c562a7723c45ee71dab1c892acaf7a608d is the merged commit for upstream PR #14705. In ScrollRestoration, it applies escapeHtml() to both JSON.stringify(storageKey || SCROLL_RESTORATION_STORAGE_KEY) and JSON.stringify(ssrKey) before embedding them in the inline script through dangerouslySetInnerHTML. This directly matches the advisory's SSR XSS condition involving untrusted getKey/storageKey values.

The commit's changeset identifies this as a react-router patch. The 7.12.0 changelog lists PR #14705, and the release tag contains this commit, matching the patched React Router version already recorded by the advisory.

Supporting sources:

The proposed advisory change only adds the fix commit reference. No changes to package ranges, patched versions, severity, CWE, or the advisory description are proposed.

@github

github commented Sep 5, 2026

Copy link
Copy Markdown
Collaborator

Hi there @brophdawg11! A community member has suggested an improvement to your security advisory. If approved, this change will affect the global advisory listed at github.com/advisories. It will not affect the version listed in your project repository.

This change will be reviewed by our Security Curation Team. If you have thoughts or feedback, please share them in a comment here! If this PR has already been closed, you can start a new community contribution for this advisory

@github-actions
github-actions Bot changed the base branch from main to HF-CYGG/advisory-improvement-9371 September 5, 2026 21:42
@brophdawg11

Copy link
Copy Markdown

I don't personally think this is necessary. It's OSS and the fix version is published so anyone who wants to know the specific commit can find it quite easily.

@advisory-database
advisory-database Bot merged commit 76d65e5 into HF-CYGG/advisory-improvement-9371 Sep 8, 2026
4 checks passed
@advisory-database

Copy link
Copy Markdown
Contributor

Hi @HF-CYGG! Thank you so much for contributing to the GitHub Advisory Database. This database is free, open, and accessible to all, and it's people like you who make it great. Thanks for choosing to help others. We hope you send in more contributions in the future!

@advisory-database
advisory-database Bot deleted the HF-CYGG-GHSA-8v8x-cx79-35w7 branch September 8, 2026 16:30
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.

3 participants