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

do not set X-Content-Security-Policy header #135

Conversation

jelhan
Copy link
Collaborator

@jelhan jelhan commented Mar 23, 2020

All major browsers support Content-Security-Policy header since a long time. Only Internet Explorer 11 does not support it. But it also only supports sandbox directive by X-Content-Security-Policy header. The sandbox is not useful at all for Ember applications cause it could only be used to disable all scripts. As an Ember applicaiton is not usable at all without JavaScript that is not a reasonable configuration.

This resolves the inconsistency between middleware used in Ember CLI's development server and FastBoot initalizier described in #133.

All major browsers support Content-Security-Policy header since a long time.
Only Internet Explorer 11 does not support it. But it also only supports
`sandbox` directive by X-Content-Security-Policy header. The sandbox is not
useful at all for Ember applications cause it could only be used to disable
all scripts. As an Ember applicaiton is not usable at all without JavaScript
that is not a reasonable configuration.
@jelhan jelhan merged commit 7e0fb24 into adopted-ember-addons:master Apr 2, 2020
@sandstrom
Copy link
Collaborator

@jelhan Great!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants