Merge ~suligap/canonical-identity-provider:new-style-middleware into canonical-identity-provider:master
Proposed by
Przemysław Suliga
Status: | Merged |
---|---|
Approved by: | Przemysław Suliga |
Approved revision: | 0036cd5864416b032f552063818e5e0d6fe2f82d |
Merge reported by: | Otto Co-Pilot |
Merged at revision: | not available |
Proposed branch: | ~suligap/canonical-identity-provider:new-style-middleware |
Merge into: | canonical-identity-provider:master |
Diff against target: |
153 lines (+18/-11) 9 files modified
django_project/settings_base.py (+1/-1) django_project/settings_test.py (+1/-1) requirements.txt (+1/-1) src/api/v20/utils.py (+2/-1) src/identityprovider/middleware/http.py (+2/-1) src/identityprovider/middleware/oopsplease.py (+3/-1) src/identityprovider/middleware/readonly.py (+4/-2) src/identityprovider/middleware/useraccount.py (+2/-1) src/identityprovider/tests/test_middleware.py (+2/-2) |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Hasan Ammar (community) | Approve | ||
Review via email: mp+384550@code.launchpad.net |
Commit message
Switch to Django 1.10 style middleware
Description of the change
https:/
In most cases this is adding the MiddlewareMixin to our custom
middleware classes. Plus a version bump for django-honeypot that brings
the new style middleware compatibility.
Other middleware used (django and other 3rd party middleware) was already
compatible.
To post a comment you must log in.
LGTM!