mirror of
https://github.com/lldap/lldap.git
synced 2026-04-25 16:25:55 +03:00
[GH-ISSUE #191] Support for the userPrincipalName attribute #74
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#74
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 @publicdesert on GitHub (Jun 15, 2022).
Original GitHub issue: https://github.com/lldap/lldap/issues/191
First of all, I really appreciate this project.
In my pull request (#190) I added an example configuration for Seafile. Since Seafile requires an unique identifier in an email-like format, I used the
mailattribute in my config. I noticed however, that this is not really ideal, since the user is able to change his personal email address himself. Therefore I was wondering if it would be possible to add support for the userPrincipalName which would return<name>@<tree name>.@nitnelave commented on GitHub (Jun 17, 2022):
Is it really that bad to log in with your email address? They're still required to be unique so there won't be a conflict.
Adding support for that attribute would complicate things more than I'd like, when we have a perfectly good solution already.
@publicdesert commented on GitHub (Jun 17, 2022):
The problem is that when using the mail attribute as an identifier, Seafile will treat you as an entirely new user if you change your email address through LLDAP.
For my personal setup I ended up using Authelia with LLDAP as the backend for Seafile and a few services that only support OAuth. Seafile's OAuth login feature is a lot more flexible then it's LDAP one. Thus I personally do not need this feature anymore, but it may be useful for others.
@nitnelave commented on GitHub (Jun 17, 2022):
Yeah, for me that's more of a Seafile problem than LLDAP.
Glad you found a solution, I'll close this issue.