users_ldap creates duplicate users because by default ldap is case insensitive.

Bug #985606 reported by Daniel Reis
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Odoo Addons (MOVED TO GITHUB)
Fix Released
Low
OpenERP R&D Addons Team 1
Therp Backports (Deprecated)
Fix Released
Low
Stefan Rijnhart (Opener)

Bug Description

Using users_ladap module with the "Create user" option activated on the "LDAP configuration".
When a user login with it's username written with different case letters, OpenERp created duplicated users.
Example: if i login first as "dreispt", and again as "DREISPT", two OpenERP users will be generated for the same LDAP user.

Related branches

Revision history for this message
Amit Parik (amit-parik) wrote :

Hello Daniel,

Yes, you are right. Ldap is not a case sensitive that's why It will make a duplicate user( I didn't check yet).

I looked your MP, It looks ok for me. It will restrict to the user with another login name(as per lower case).

You have already put a MP for this, So our Core team will review it.

Thanks for contribution!

Changed in openobject-addons:
assignee: nobody → OpenERP R&D Addons Team 1 (openerp-dev-addons1)
importance: Undecided → Low
status: New → Fix Committed
summary: - users_ldap creates duplicate users
+ users_ldap creates duplicate users because by default ldap is case
+ insensitive.
no longer affects: therp-addons
Changed in therp-backports:
milestone: none → 6.1-maintenance
status: New → In Progress
importance: Undecided → Low
assignee: nobody → Stefan Rijnhart (Therp) (stefan-therp)
Revision history for this message
Daniel Reis (dreis-pt) wrote :

Stefan's branch provides a more proper solution, so I deleted the original merge proposals.

Changed in therp-backports:
status: In Progress → Fix Released
Revision history for this message
Fabien (Open ERP) (fp-tinyerp) wrote :

If I am not wrong, depending on your LDAP configuration, an attribute in LDAP can be case sensitive.
I would suggest you to change your LDAP attribute.

Changed in openobject-addons:
status: Fix Committed → Invalid
Revision history for this message
Stefan Rijnhart (Opener) (stefan-opener) wrote :

Hi Fabien,

it looks like you are wrong. The 'uid' attribute for computer logins is case insensitive by specification.

http://tools.ietf.org/html/rfc4519#page-5

Cheers,
Stefan.

Changed in openobject-addons:
status: Invalid → Confirmed
Revision history for this message
Fabien (Open ERP) (fp-tinyerp) wrote :

my mistake, sorry, you are right.

Changed in openobject-addons:
status: Confirmed → Fix Released
Lara (Therp) (lfreeke)
Changed in therp-backports:
milestone: 6.1-maintenance → fixed-after-61
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.