[GH-ISSUE #125] uid search is case sensitive #52

New issue

Closed

opened 2026-02-27 08:14:58 +03:00 by kerem · 3 comments

kerem commented

2026-02-27 08:14:58 +03:00

Owner

Originally created by @crobibero on GitHub (Mar 13, 2022).
Original GitHub issue: https://github.com/lldap/lldap/issues/125

All fields are currently case sensitive, where they should be case insensitive. The password field is expected to be case sensitive.

RFC (uid deep link): https://datatracker.ietf.org/doc/html/rfc4519#section-2.39

      ( 0.9.2342.19200300.100.1.1 NAME 'uid'
         EQUALITY caseIgnoreMatch
         SUBSTR caseIgnoreSubstringsMatch
         SYNTAX 1.3.6.1.4.1.1466.115.121.1.15 )

Originally created by @crobibero on GitHub (Mar 13, 2022). Original GitHub issue: https://github.com/lldap/lldap/issues/125 All fields are currently case sensitive, where they should be case insensitive. The password field is expected to be case sensitive. RFC (uid deep link): https://datatracker.ietf.org/doc/html/rfc4519#section-2.39 ``` ( 0.9.2342.19200300.100.1.1 NAME 'uid' EQUALITY caseIgnoreMatch SUBSTR caseIgnoreSubstringsMatch SYNTAX 1.3.6.1.4.1.1466.115.121.1.15 ) ```

kerem closed this issue

2026-02-27 08:14:59 +03:00

kerem commented

2026-02-27 08:14:59 +03:00

Author

Owner

@nitnelave commented on GitHub (Mar 26, 2022):

At the risk of slightly breaking from the RFC, I'd like to keep most of the fields case sensitive apart from the username. The others are only used for display anyway.

@nitnelave commented on GitHub (Mar 26, 2022): At the risk of slightly breaking from the RFC, I'd like to keep most of the fields case sensitive apart from the username. The others are only used for display anyway.

kerem commented

2026-02-27 08:14:59 +03:00

Author

Owner

@crobibero commented on GitHub (Mar 26, 2022):

That works for me, thanks!

@crobibero commented on GitHub (Mar 26, 2022): That works for me, thanks!

kerem commented

2026-02-27 08:14:59 +03:00

Author

Owner

@nitnelave commented on GitHub (Mar 26, 2022):

Alright, great! One small note: if you have users that have already signed up with uppercase logins, you'll need to lowercase them in the DB:

UPDATE users SET user_id = LOWER(user_id);

@nitnelave commented on GitHub (Mar 26, 2022): Alright, great! One small note: if you have users that have already signed up with uppercase logins, you'll need to lowercase them in the DB: ```sql UPDATE users SET user_id = LOWER(user_id); ```

kerem referenced this issue

2026-02-27 08:17:49 +03:00

[PR #52] [MERGED] config: Add a minimum password length #512