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

AccessControlAllowHeader's iter() might not behave properly #60

Open
AlaricSenat opened this issue Dec 19, 2019 · 1 comment
Open

AccessControlAllowHeader's iter() might not behave properly #60

AlaricSenat opened this issue Dec 19, 2019 · 1 comment

Comments

@AlaricSenat
Copy link

According to CORS:
Access-Control-Allow-Headers: "Access-Control-Allow-Headers" ":" #field-name
You can find the definition of the construct #rule in the http1.1 rfc.

Then why calling iter() on such a header: "AccessControlAllowHeaders(",x-print") (note the comma) returns an empty iterator instead of something like "", "x-print" ?

This behavior was implemented by 026949a but is not consistent with all the other header types having a #field-name as values (AccessControlExposeHeaders for instance).

@pshaughn
Copy link

The 'Strange allowheaders (case insensitive)' case of https://github.com/web-platform-tests/wpt/blob/master/cors/request-headers.htm expects a leading comma to be accepted. I notice that the parser is fine with the consecutive commas in the middle; it's only the leading one that's causing problems.

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

2 participants