Skip to content

Conversation

@herndlm
Copy link
Contributor

@herndlm herndlm commented Jun 6, 2018

In base_js_html.twig eval is used but the ContentSecurityPolicyHandler never adds 'unsafe-eval' to the nonce which breaks the profiler with very strict CSPs

Q A
Branch? 3.2
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets #...
License MIT
Doc PR symfony/symfony-docs#...

I was using the profiler bundle with the very strict CSP "default-src 'self'; script-src 'self';" and the browser was not loading the toolbar because of CSP violations.

I honestly don't know if this is the best way to fix them / handle unsafe-eval, but this is working fone for me and maybe @romainneutron can help me out

herndlm added 2 commits June 6, 2018 13:20
In base_js_html.twig eval is used but the ContentSecurityPolicyHandler never adds 'unsafe-eval' to the nonce which breaks the profiler with very strict CSPs
@stof
Copy link
Member

stof commented Jun 6, 2018

Please avoid reducing the indentation by 1 space here. It makes the diff impossible to review. We use 4 spaces per indentation level

@stof
Copy link
Member

stof commented Jun 6, 2018

and btw, bug fixes are not allowed in 3.2 anymore. This branch is not maintained: http://symfony.com/roadmap/3.2

@herndlm
Copy link
Contributor Author

herndlm commented Jun 6, 2018

sorry for that. I created #27525

@herndlm herndlm closed this Jun 6, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants