search

susundberg / django-auth-ldap-ad

Django authentication backend for LDAP with Active Directory
GNU General Public License v2.0
76 stars 27 forks source link

Port to Python3 #11

Open nanonyme opened 7 years ago

nanonyme commented 7 years ago

This is more a result of investigation than feature request. At this point ldap3 supports both Python2 and Python3 so I'm proposing following migration plan (and am hoping to be able to further it myself) 1) Migrate from ldap-python to ldap3 without caring about Python3 compat 2) Add test harness for Python3 3) Fix test and code until everything passes with both Python2 and Python3 I'm proposing minimum requirements to be set to later builds of 2.7 and 3.3 branches as sensible since syntax overlap is significant

susundberg commented 7 years ago

Hi!

The plan sounds good, need (for python3 support) is real, and the ldap3 library seems well established. All go!

Anyway, such major update will be new release (lets call that 2.0), so if any issues with real world server arrives, one can always fall back to old library version.

I am currently very out of time with this project, so if you just find some time (or you find somebody with time:), project would be more than happy.

Cheers, Pauli

nanonyme commented 7 years ago

Documenting here that relevant mock test info for ldap3 is currently available at http://ldap3.readthedocs.io/mocking.html

nanonyme commented 7 years ago

Currently blocked by cannatag/ldap3#347 ; MOCK_ASYNC maps better with how this library used to use python-ldap