-
Notifications
You must be signed in to change notification settings - Fork 186
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
Request for Comments: Implement RFC 5987 #27
Comments
+1, let's do it. |
Well, this should have been implemented a long time ago :) However, it should be in requests not toolbelt. |
@piotr-dobrogost Disagree. Requests should not be checking encodings or assuming that endpoints support RFC 5234. Many don't, and that kind of failure is really nasty to work around. Better to have it be an explicit opt-in. Remember the commitment of the requests project to the 90% use-case. =D This project is for the remaining 10%. |
Actually, I'd like to think of this supporting 90% of that 10% ;). |
@piotr-dobrogost the other thing is that we reserve the right to eventually move elements of the toolbelt into the core library if Kenneth wants to. The toolbelt is licensed the same as requests for just that purpose. |
So contrary to what @piotr-dobrogost seemed to hint at in kennethreitz/requests#1926 this does not affect the actual value of the header. This just allows you to add extra information to the header in a different encoding. |
I really don't get what you meant to say above. |
RFC 5987 does not allow for you to do the following:
It does, however allow you to do this:
|
RFC 5987 provides a way of encoding arbitrary character encodings in ISO-8859-1 for inclusion in header values compliant with RFC 2616. Recently there have been issues opened by people expecting unicode to be usable in header values (see: https://github.com/kennethreitz/requests/issues/1926 and httpie/cli#212). Since requests is following the specification, it makes sense to me that the toolbelt could provide some functionality for dealing with this.
Thoughts @Lukasa, @piotr-dobrogost, or @shazow?
The text was updated successfully, but these errors were encountered: