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

Not working in FF Developer edition #1

Closed
SleepProgger opened this issue Oct 26, 2015 · 6 comments
Closed

Not working in FF Developer edition #1

SleepProgger opened this issue Oct 26, 2015 · 6 comments

Comments

@SleepProgger
Copy link
Contributor

Just a little heads up:
I played a bit with the project and noticed it is working in the current FF release (not as nicely as in chrome but it works with adding the CSP header via PHP), but didn't work in the FF Developer edition.

In fact it looks like dev edition ignores the CSP rule completely as it happily displays images from an HTTPS source.

@diracdeltas
Copy link
Owner

In the developer tools console, do you see the CSP rule being sent correctly? If so, that sounds like a FF bug.

Could you try changing the CSP syntax to img-src http:?

@jomo
Copy link

jomo commented Oct 26, 2015

From security csp in the Developer Toolbar, it also shows up like that in response headers:
CSP screenshot


it happily displays images from an HTTPS source

Can confirm, was able to append that HTTPS screenshot to the page:

https screenshot


To be fair, it doesn't seem to make a lot of sense to restrict images to insecure URLs, but I would agree that's a bug.

@diracdeltas
Copy link
Owner

I also confirmed this, and it's definitely a bug according to the CSP spec. Testing it out in csptester.io, it looks like Firefox Dev Edition does not ignore 'img-src https:' but does ignore 'img-src http:'.

@diracdeltas
Copy link
Owner

@diracdeltas
Copy link
Owner

Also see w3c/webappsec-csp#25

@diracdeltas
Copy link
Owner

Update: FF and Chrome have decided that ignoring the http-only CSP directive is correct. As a workaround, Sniffly Firefox uses crbug 436451 for sniffing. Closing for now; please reopen if it still doesn't work!

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

No branches or pull requests

3 participants