-
Notifications
You must be signed in to change notification settings - Fork 2.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
nonce support for defer inline scripts #5156
Comments
Looks like this is a React issue as well: facebook/react#26026 But Deferred will need these as well I think. |
React actually does support nonce, so we just need support for this in Remix's deferred-related scripts. |
This issue has been automatically marked stale because we haven't received a response from the original author in a while 🙈. This automation helps keep the issue tracker clean from issues that are not actionable. Please reach out if you have more information for us or you think this issue shouldn't be closed! 🙂 If you don't do so within 7 days, this issue will be automatically closed. |
Still relevant. |
I'm hitting this issue too. I wasn't successful in writing any tests for |
Fixed by #6389 and will be available in the next release |
🤖 Hello there, We just published version Thanks! |
🤖 Hello there, We just published version Thanks! |
🤖 Hello there, We just published version Thanks! |
🤖 Hello there, We just published version Thanks! |
This patch provides the `nonce` option to the React `renderToPipeableStream` function so that React will add it to scripts dynamically injected (e.g. the `<Suspense>` inline scripts). Ref: https://react.dev/reference/react-dom/server/renderToPipeableStream#parameters Ref: facebook/react#26026 (comment) Ref: remix-run/remix#5156 (comment)
Because security please :)
The text was updated successfully, but these errors were encountered: