Skip to content
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

Allows setting nonce on generated style tags #376

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

swaff
Copy link

@swaff swaff commented Apr 4, 2018

Uses a feature of webpack to extract a set nonce value on the generated
style tags to allow users to maintain a Content Security Policy that
does not need to use unsafe-inline for the styles.

Updates the makeStyleTag function to optionally add the nonce
attribute if the __webpack_nonce__ value is set.

Adds documentation of the feature at docs/nonce.md

Uses a feature of webpack to extract a set nonce value on the generated
style tags to allow users to maintain a Content Security Policy that
does not need to use `unsafe-inline` for the styles.

Updates the `makeStyleTag` function to optionally add the `nonce`
attribute if the `__webpack_nonce__` value is set.

Adds documentation of the feature at docs/nonce.md
@frontendphil
Copy link

@threepointone since this is pretty crucial for us, how can we help speed up getting this merged? In general, I see quite some useful PRs in the repo, that are stalling.

@frontendphil
Copy link

frontendphil commented Sep 25, 2018

Hey @swaff, since we at Signavio believe in this library we want to continue developing it further. Therefore, we've forked it and will release it as signavio-glamor. It would be awesome if you could direct your PR against our fork so that we can include it!

https://github.com/signavio/glamor

@swaff
Copy link
Author

swaff commented Sep 25, 2018

@frontendphil That is done for you now, good luck with the project.

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.

None yet

2 participants