Gitlab::LDAP::Person uses LDAP attributes configuration
What does this MR do?
We allow users to configure LDAP attribute preferences. For example,
email can be configured to use mail
, email
and userPrincipalName
,
falling through to the next until a value is found. Prior to this
change, Gitlab::LDAP::Person did not honor this configuration. Now,
the class will honor name
and mail
configuration. It does not
handle username
, or fallback to first_name
+ last_name
in the
absence of name
.
Closes #26290 (closed)
cc/ @MrChrisW
Merge request reports
Activity
added 1 commit
- cb4525e8 - Gitlab::LDAP::Person uses LDAP attributes configuration
assigned to @rspeicher
- Resolved by Drew Blessing
- Resolved by Drew Blessing
- Resolved by Drew Blessing
- Resolved by Drew Blessing
added 1 commit
- f6de53e0 - Gitlab::LDAP::Person uses LDAP attributes configuration
added 1 commit
- 5dd29984 - Gitlab::LDAP::Person uses LDAP attributes configuration
added 1 commit
- babb7d52 - Gitlab::LDAP::Person uses LDAP attributes configuration
- Resolved by Drew Blessing
@dblessing LGTM. Feel free to change milestone and add ~"Pick into Stable" if you think this needs to be in an 8.15 patch.
changed milestone to %8.16
enabled an automatic merge when the pipeline for babb7d52 succeeds
Thanks @rspeicher
changed milestone to %8.15
mentioned in commit 115aac77
mentioned in issue #26163 (closed)
mentioned in commit dbbac3e9
mentioned in merge request !8465 (merged)