mirror of
https://github.com/lldap/lldap.git
synced 2026-04-25 00:05:50 +03:00
[GH-ISSUE #999] [BUG] unable to search using firstname #362
Labels
No labels
backend
blocked
bug
cleanup
dependencies
docker
documentation
duplicate
enhancement
enhancement
frontend
github_actions
good first issue
help wanted
help wanted
integration
invalid
ldap
pull-request
question
rust
rust
tests
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/lldap-lldap#362
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @ggodlewski on GitHub (Oct 13, 2024).
Original GitHub issue: https://github.com/lldap/lldap/issues/999
Describe the bug
I'm unable to search using first name.
According to https://github.com/lldap/lldap/issues/998#issuecomment-2408959241 this should work?
I also tried
givenName:I do have user with following name:
@gabay commented on GitHub (Oct 19, 2024):
I took a look at what happens:
In short - the first (and last) name is stored as a case-sensitive blob in the table "user_attributes"
While preforming the search - name is transformed to a lower-case blob, resulting in a mismatch when comparing the blobs...
To verify this - you could set first name to "grzegorz" (lowercase) and see that your query works regardless of the query case :)
A quick hack would be making all names lower-case all the time.
@nitnelave commented on GitHub (Oct 26, 2024):
Yeah, that'll require a bit more design work. Currently, attributes are not stored in an easily-searchable format, especially if the case is different.
I can make a small fix that doesn't lowercase the search filter, so at least it'll find if you have the exact spelling and casing, but it's not ideal.
@broeng commented on GitHub (Jan 13, 2025):
Just fyi, @nitnelave the small fix you suggest, is actually what I have implemented in !1074.
I hit this issue with Samba reading sambaSID ID's from LLDAP, and expecting to use them for lookup later. They also contain uppercase characters.
@broeng commented on GitHub (Jan 22, 2025):
Should be fixed with merge of PR #1074