-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Content-Type lookup is case sensitive #129
Comments
hadrien
added a commit
to hadrien/chalice
that referenced
this issue
Sep 30, 2016
hadrien
added a commit
to hadrien/chalice
that referenced
this issue
Sep 30, 2016
hadrien
added a commit
to hadrien/chalice
that referenced
this issue
Sep 30, 2016
jamesls
pushed a commit
that referenced
this issue
Oct 10, 2016
jamesls
added a commit
that referenced
this issue
Oct 10, 2016
* hadrien-fix/129-content-type-lookup-is-case-sensitive: Ensure to_dict() is JSON serializable Update changelog with latest changes Read-only case insensitive mapping for headers. Fix issue #129
Closing, #130 has been merged. |
Thanks so much for the work on this! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Using
fetch()
method from javascript and it does a lowercase operation before it send out my header. That meansContent-Type
becomescontent-type
.This problem was noted in the code already:
https://github.com/awslabs/chalice/blob/master/chalice/app.py#L72-L75
Turns out this problem happened recently with the addition of 1a2ae87.
I suggest just checking for both:
Headers should be case-insensitive, so perhaps the class should lowercase all the headers as it reads them?
The text was updated successfully, but these errors were encountered: