Skip to content
This repository has been archived by the owner on Jan 9, 2019. It is now read-only.

Add inline_js Method #65

Open
darkwing opened this issue Jun 6, 2014 · 3 comments
Open

Add inline_js Method #65

darkwing opened this issue Jun 6, 2014 · 3 comments

Comments

@darkwing
Copy link

darkwing commented Jun 6, 2014

Is there any reason to avoid inline JavaScript? Could save a request for tiny JS file content.

@jsocol
Copy link
Owner

jsocol commented Jun 6, 2014

I'm actually surprised we didn't add it in #59

@cvan
Copy link
Collaborator

cvan commented Jun 11, 2014

FWIW, inline JS is a CSP violation and will not execute. But we should still do this.

@jsocol
Copy link
Owner

jsocol commented Jun 11, 2014

Well it's only a CSP violation if the policy forbids it. For sites that don't allow user-generated content it's not at all unreasonable. --

Sent from my phone, please excuse the occasional typo.

On Wed, Jun 11, 2014 at 5:49 PM, Christopher Van [email protected]
wrote:

FWIW, inline JS is a CSP violation. But we should still do this.

Reply to this email directly or view it on GitHub:
#65 (comment)

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

No branches or pull requests

3 participants