LDAPUserFolder authenticates against wrong attribute

Bug #650411 reported by DavidJB
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Products.LDAPUserFolder
Won't Fix
Medium
Unassigned

Bug Description

The product uses the attribute from 'Login Name Attribute' (_login_attr from the LDAPUserFolder object) to check authentication requests, rather than 'User ID Attribute' (_uid_attr) as one could possibly expect.

Within Plone, the distinction between these two attributes is evident: the Canonical Name or 'Login Name' is typically mapped to a user's actual Full Name, whilst the User ID is self explanatory.

This seems to be a bug to me, as it is an incorrect mapping.

To fix this problem, edit Line 774 of Products.LDAPUserFolder/Products/LDAPUserFolder/LDAPUserFolder.py and change 'self._login_attr' to 'self._uid_attr'.

This causes LDAPUserFolder to start comparing the username entered with the user ID attribute rather than the user's CN.

Tags: bug
Revision history for this message
Jens Vagelpohl (dataflake-deactivatedaccount-deactivatedaccount) wrote :

The login attribute is *exactly* the right attribute to use, why else do you think it is even there? Its meaning, as the name clearly suggest, is "this is the attribute to compare user logins against".

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.