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

CSP: clarify whitespace characters #498

Closed
michaelficarra opened this issue Oct 6, 2015 · 3 comments
Closed

CSP: clarify whitespace characters #498

michaelficarra opened this issue Oct 6, 2015 · 3 comments

Comments

@michaelficarra
Copy link
Contributor

See my comment in #495 (comment).

4.2.1 separates source expressions on U+0020 (space), U+0009 (tab), U+000A (line feed), U+000C (form feed), and U+000D (carriage return), and the source-list grammar separates source expression on WSP, which is only U+0020 (space) and U+0009 (tab).

I think we should just replace usage of RFC 5234 WSP (the latter usage above) with HTML5's space characters (the former usage above).

@hillbrad
Copy link
Contributor

hillbrad commented Oct 6, 2015

As part of a HTTP header, though, we can't allow line feeds or carriage
returns.

On Tue, Oct 6, 2015 at 5:57 AM Michael Ficarra [email protected]
wrote:

See my comment in #495 (comment)
#495 (comment).

4.2.1 separates source expressions on U+0020 (space), U+0009 (tab), U+000A
(line feed), U+000C (form feed), and U+000D (carriage return), and the
source-list grammar separates source expression on WSP, which is only
U+0020 (space) and U+0009 (tab).

I think we should just replace usage of RFC 5234 WSP (the latter usage
above) with HTML5's space characters (the former usage above).


Reply to this email directly or view it on GitHub
#498.

@shekyan
Copy link

shekyan commented Oct 6, 2015

CSP spec should be transport agnostic and inputs containing LF and CR would be restricted by HTTP. Consider the case of meta-tag as a transport and HTML entity representing a new line.

@mikewest
Copy link
Member

mikewest commented Oct 7, 2015

This issue was moved to w3c/webappsec-csp#5

@mikewest mikewest closed this as completed Oct 7, 2015
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

4 participants