[PATCH 01/19] settings: Enable 'SessionAuthenticationMiddleware'

Sean Farley sean at farley.io
Thu May 11 06:11:00 AEST 2017


Stephen Finucane <stephen at that.guru> writes:

> This raises warnings for Django 1.8 and is mandatory in Django 1.10. It
> provides a helpful feature, invalidating a user's session when their
> password is changed, and can/should be enabled.
>
> This resolves all issues with Django 1.8.
>
> Signed-off-by: Stephen Finucane <stephen at that.guru>
> ---
>  patchwork/settings/base.py | 12 +++++++++---
>  1 file changed, 9 insertions(+), 3 deletions(-)
>
> diff --git a/patchwork/settings/base.py b/patchwork/settings/base.py
> index 5f6aebe..d82c41d 100644
> --- a/patchwork/settings/base.py
> +++ b/patchwork/settings/base.py
> @@ -35,10 +35,16 @@ MIDDLEWARE_CLASSES = [
>  ]
>  
>  if django.VERSION >= (1, 7):
> -    MIDDLEWARE_CLASSES.append(
> -        'django.contrib.admindocs.middleware.XViewMiddleware')
> +    MIDDLEWARE_CLASSES += [
> +        'django.contrib.auth.middleware.SessionAuthenticationMiddleware',
> +    ]
> +
> +if django.VERSION >= (1, 7):
> +    MIDDLEWARE_CLASSES += [
> +        'django.contrib.admindocs.middleware.XViewMiddleware'
> +    ]

Should this be consolidated with the 'if django.VERSION >= (1, 7)'
above?
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 800 bytes
Desc: not available
URL: <http://lists.ozlabs.org/pipermail/patchwork/attachments/20170510/d32b61aa/attachment.sig>


More information about the Patchwork mailing list