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

Check header names and status for line feed/carriage return. Fixes #122 #124

Merged
merged 1 commit into from Mar 26, 2016

Conversation

@jamadden
Copy link
Contributor

jamadden commented Mar 19, 2016

No description provided.

@bertjwregeer
Copy link
Member

bertjwregeer commented Mar 19, 2016

Yay! Will review and merge. Thank you!

@jamadden
Copy link
Contributor Author

jamadden commented Mar 24, 2016

gunicorn merged a similar patch recently.

@bertjwregeer
Copy link
Member

bertjwregeer commented Mar 24, 2016

I haven't had time to review it, verify it all works correctly, and get a new release out. Tomorrow is good Friday so I will have the day off to get this done :-)

@jamadden
Copy link
Contributor Author

jamadden commented Mar 24, 2016

I'm sorry, I wasn't trying to pressure anybody, I was just trying to tie things together for the historical context.

@bertjwregeer
Copy link
Member

bertjwregeer commented Mar 24, 2016

@jamadden Ah, no worries :-) I was simply adding a timeline for when I hope this to be completed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.