django-password-policies icon indicating copy to clipboard operation
django-password-policies copied to clipboard

Broken in Django 1.10

Open djangoat opened this issue 8 years ago • 12 comments

Changes in Django 1.10 means MIDDLEWARE_CLASSES now becomes MIDDLEWARE.

password_policies.middleware.PasswordChangeMiddleware now broken:

in load_middleware
    mw_instance = middleware(handler)
TypeError: object() takes no parameters

https://docs.djangoproject.com/en/1.10/topics/http/middleware/#upgrading-pre-django-1-10-style-middleware

All it seems to need is:

from django.utils.deprecation import MiddlewareMixin

class PasswordChangeMiddleware(MiddlewareMixin):

Tested, and seems fine.

djangoat avatar Sep 04 '16 09:09 djangoat

See PR #59 comments...

tarak avatar Oct 15 '16 21:10 tarak

Hey @tarak, I love your app, any chances of supporting Django 1.10 soon? Thanks!

mpasternak avatar Mar 10 '17 08:03 mpasternak

Any plans to merge pull request https://github.com/tarak/django-password-policies/pull/63 to address this issue?

abarresi-abamis avatar Apr 03 '17 17:04 abarresi-abamis

What's up here @tarak ? We are looking to migrate to 1.11...

aboudreault avatar May 15 '17 20:05 aboudreault

I suggest to update package and follow https://www.djangoproject.com/download/

Supported Django versions: Django 1.8 LTS, 1.10, 1.11 LTS. For now looks like this package is broken in 66,67% of supported configurations...

lorddaedra avatar May 23 '17 09:05 lorddaedra

There's pull request for that. @tarak , if you're no longer interested in maintaining the package perhaps it would be wise to consider passing the maintainership to someone... unless someone forks it

mpasternak avatar Jun 19 '17 08:06 mpasternak

Just to let you know, I've e-mailed @tarak and asked him to give me commit privs to the repo and to the PyPI package, I'm willing to maintain it.

mpasternak avatar Aug 30 '17 22:08 mpasternak

@mpasternak @tarak What is the status of this?

DanEEStar avatar Nov 07 '17 13:11 DanEEStar

I’ve received no reply at all.

mpasternak avatar Nov 07 '17 13:11 mpasternak

Django 2.0 requires on_delete argument to ForeignKeys, so I guess I'm going to prepare a fix in my branch.

EDIT: here you go https://github.com/tarak/django-password-policies/pull/64

mpasternak avatar Dec 09 '17 01:12 mpasternak

This project is dead guys, @tarak is just not professional enough to mention us he doesn't want to work on it anymore and too lazy to give the maintainer role to someone else. Seriously, just fork this project and we will all adopt it.

aboudreault avatar Dec 09 '17 06:12 aboudreault

Hi,

I have forked the project and I continue the work in my fork: https://github.com/iplweb/django-password-policies-iplweb

Feel free to report any issues, patches, docs improvements also badly needed.

I'm using this in a commercial project so I guess I'll maintain my fork for a few years at least.

mpasternak avatar Dec 10 '17 21:12 mpasternak