
david got a fish trophy!
Update to modern MIDDLEWARE setting.
Review Request #11911 — Created Jan. 7, 2022 and submitted
Information | |
---|---|
david | |
Review Board | |
release-4.0.x | |
Reviewers | |
reviewboard | |
Django 1.10 introduced some changes to the way that middleware works.
Instead of having separateprocess_request
andprocess_response
methods, middleware is now a callable which chains together. This
simplifies most cases to the point where most middleware can be
implemented as a nested function rather than a class. Opting into this
new behavior is done by changing the settings key from
MIDDLEWARE_CLASSES
toMIDDLEWARE
.This change updates Review Board to use the new setting, and the
built-in middlewares to use the new pattern. As part of this, I needed
to upgrade django-cors-headers to a newer version. I've chosen the last
version that still supports Python 2.7.
- Ran unit tests.
- Smoke tested the Review Board UI.
Summary | |
---|---|
Description | From | Last Updated |
---|---|---|
Seems the class should keep the bulk of the docstring in this case. |
|
-
-
reviewboard/site/middleware.py (Diff revision 1) Seems the class should keep the bulk of the docstring in this case.
Commits: |
|
|||||||||
---|---|---|---|---|---|---|---|---|---|---|
Diff: |
Revision 2 (+432 -166) |