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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add ability to provide a nonce for style injection #387

Merged
merged 1 commit into from
Jun 14, 2022

Conversation

seanparmelee
Copy link
Contributor

Hello! 馃憢
This is somewhat related to #139 in that we're also using a content security policy that's preventing the inline styles from being applied.

I realize that one way around this is to pass the disableAutomaticStylesInjection prop and add the styles manually, but I was curious if you'd be open to adding a nonce prop so that the injected style tag can have its nonce attribute set and we can continue to use automatic style injection along with our CSP.

@codesandbox-ci
Copy link

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 3205ae7:

Sandbox Source
react-easy-crop Configuration

@ValentinH ValentinH added enhancement New feature or request minor Increment the minor version when merged labels Jun 14, 2022
@ValentinH
Copy link
Owner

Thanks for this contribution. I understand that this could be useful for some people and avoid having to opt-out of the automatic style injection. Let's merge this!

@ValentinH ValentinH merged commit f97472e into ValentinH:main Jun 14, 2022
@github-actions
Copy link

馃殌 PR was released in v4.4.0 馃殌

@github-actions github-actions bot added the released This issue/pull request has been released. label Jun 14, 2022
@seanparmelee seanparmelee deleted the nonce branch June 14, 2022 16:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request minor Increment the minor version when merged released This issue/pull request has been released.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants