-
-
Notifications
You must be signed in to change notification settings - Fork 6.9k
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
Use default reason phrases from HTTP standard. #2764
Conversation
Fixes #2763. This looks good. Why Travis no run? |
@carltongibson I was wondering that same thing. It's not even under pull requests. |
@jpadilla Nice link. Not the one you meant though. :-) |
@carltongibson ugh fixed, at least it wasn't something embarrassing. |
Indeed. :-) My Travis-Foo isn't up to knowing how to shout at it. |
This SO answer says close the PR and re-open to trigger the build. Slightly heavy handed but I guess it'd work. (The web hook must have fizzled I guess.) |
@carltongibson yeap that worked, thanks. |
Well let's have this one then. Thanks @jpadilla! |
Use default reason phrases from HTTP standard. Fixes #2763
Ticket: https://code.djangoproject.com/ticket/24137
Pull Request: django/django#3902
django.core.handlers.wsgi.STATUS_CODE_TEXT
has been removed in favor of using Python's stdlib instead,http.client.responses
for Python 3 andhttplib.responses
for Python 2.