Skip to content

Allows setting nonce on generated style tags#376

Open
swaff wants to merge 1 commit into
threepointone:masterfrom
swaff:assign-nonce-from-webpack-variable
Open

Allows setting nonce on generated style tags#376
swaff wants to merge 1 commit into
threepointone:masterfrom
swaff:assign-nonce-from-webpack-variable

Conversation

@swaff
Copy link
Copy Markdown

@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
Copy Markdown

@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
Copy Markdown

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
Copy Markdown
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.

2 participants