[GHSA-8v8x-cx79-35w7] React Router SSR XSS in ScrollRestoration - #9371
Conversation
|
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 |
|
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. |
76d65e5
into
HF-CYGG/advisory-improvement-9371
|
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! |
Updates
Comments
Add the missing upstream React Router v7 fix commit for GHSA-8v8x-cx79-35w7.
Commit
c89c32c562a7723c45ee71dab1c892acaf7a608dis the merged commit for upstream PR #14705. InScrollRestoration, it appliesescapeHtml()to bothJSON.stringify(storageKey || SCROLL_RESTORATION_STORAGE_KEY)andJSON.stringify(ssrKey)before embedding them in the inline script throughdangerouslySetInnerHTML. This directly matches the advisory's SSR XSS condition involving untrustedgetKey/storageKeyvalues.The commit's changeset identifies this as a
react-routerpatch. 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.