mirror of
https://github.com/lldap/lldap.git
synced 2026-04-25 16:25:55 +03:00
[GH-ISSUE #728] [BUG] lldap website auth's username is case sensitive, while password reset is not #264
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#264
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 @evrardjp on GitHub (Nov 1, 2023).
Original GitHub issue: https://github.com/lldap/lldap/issues/728
Describe the bug
When logging in, I need to provide a username in a case sensitive manner (makes sense to me).
Yet, if I go to password reset page, I add a capital letter for the login to recover, and it will work.
If this is intentional, it can definitely confuse people "Why does the password reset work, but auth does not?"
To Reproduce
Steps to reproduce the behavior:
Expected behavior
Step 3 should not have worked without the correct case.
Logs
If you want logs I can provide some.
Additional context
Is that expected?
@lordratner commented on GitHub (Dec 25, 2023):
When you create a user, it is not case sensitive. If you already have user "johndoe" you will not be able to create "JohnDoe"
I think everything should be case-insensitive (for usernames obviously) and force it to lowercase in the webui. Easier that way.
@nitnelave commented on GitHub (Aug 19, 2024):
I'm not sure whether that was actually the case when the issue was opened, but it doesn't seem to be the case anymore. I've been pretty careful to use some type-level guarantees to ensure case insensitivity.