Skip to content
This repository has been archived by the owner on Jan 9, 2018. It is now read-only.

Use get_internal_wsgi_application in Django 1.4 #36

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from
Open

Use get_internal_wsgi_application in Django 1.4 #36

wants to merge 2 commits into from

Conversation

streeter
Copy link

Mimic the runserver command that ships in Django 1.4 and up more
closely. By default, that command gets the WSGI handler by calling
get_internal_wsgi_application(). Without this change, the runserver
command ignores the WSGI_APPLICATION setting in Django 1.4.
By ignoring the setting, the runserver command will not apply any
WSGI middleware in the wsgi.py file.

If get_internal_wsgi_application is not available for import, revert back
to the previous WSGIHandler() function.

Mimic the runserver command that ships in Django 1.4 and up more
closely. By default, that command gets the WSGI handler by calling
`get_internal_wsgi_application()`. Without this change, the `runserver`
command ignores the `WSGI_APPLICATION` setting in Django 1.4
@travisbot
Copy link

This pull request fails (merged 4838550 into b1540da).

@travisbot
Copy link

This pull request fails (merged 80e7a0a into b1540da).

@streeter
Copy link
Author

So I ran make test on all three supported Django versions, and they passed. It looks like the flake8 tests are failing, but not for reasons in this this pull request.

@jezdez
Copy link
Owner

jezdez commented Aug 29, 2012

Yeah, this is expected, flake8 got much more pedantic recently, I'll fix this before merging.

@streeter
Copy link
Author

You don't have to pull in 80e7a0a as it was an attempt to quell (what I thought) was the flake8 error.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants