-
-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Utf8 headers #68
Utf8 headers #68
Conversation
…ng/trailing whitespaces in the value
Lgtm, could you just move value.strip() under name.strip() |
Add support for utf-8 values in HTTP headers
Done. |
I think we should integrate multidict first. My recent exceptions change brakes backward compatibility. I prefer to brake everything at the same time :) |
@fafhrd91 sounds good for me. |
when do you plane to make it? Anyway we can probably release gunicorn and On Sat, Jun 7, 2014 at 5:02 PM, Andrew Svetlov notifications@github.com
|
in most cases current aiohttp release should work :) On Jun 7, 2014, at 8:39 AM, Benoit Chesneau notifications@github.com wrote:
|
See also #67
@fafhrd91 I'll push it in master if you have no objections