-
-
Notifications
You must be signed in to change notification settings - Fork 4.1k
Fix default charset in FormUrlEncoded #5920
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
Conversation
2f56a08 to
644e2b0
Compare
|
I wrote a custom BodyParser based these changes and that works properly. |
|
Isn't Edit: but looks like there are other parsers ( |
|
@marcospereira RFC 2616 defines defaults for You can find a decoding algorithm for |
|
Thanks for the explanation, @gmethvin. Also, I think we can add a test case to check cases where the default charset is used and also when the request charset is specified. |
adaf9e2 to
fbba6e7
Compare
|
@marcospereira tests added |
|
@marcospereira Can you backport to 2.5.x? |
|
@mkurz already doing it. ;-) |
|
👍 |
|
Backport to 2.5.x: 6d63a8b |
Fixes #5919.
Requires backport to 2.5.x.