ensky / taiga-contrib-ldap-auth

Taiga plugin for LDAP authentication
http://taiga.io
GNU Affero General Public License v3.0
54 stars 37 forks source link

LDAP_SEARCH_PROPERTY = 'mail' not working #37

Open leonidas-o opened 7 years ago

leonidas-o commented 7 years ago

Using LDAP_SEARCH_PROPERTY = 'uid' is working without any issues, just after setting LDAP_SEARCH_PROPERTY = 'mail', log in is not possible anymore. We do not use registration, or any invitation emails. Just the pure log in over ldap.

veox commented 7 years ago

This is an issue with the way the plugin searches for LDAP entries and then matches them to local DB entries.

I've fixed this (EDIT: as a side-effect) in my fork here, but won't submit a PR yet. It touches 50% of all lines, and that mandates a thorough explanation (which I'm too tired to write ATM). Also, it changes many configuration parameter names (which makes it backward-incompatible).

@ensky, are you interested in a PR? I'd prioritise submitting this week then. (EDIT: referenced below as #47.)