feat(auth): add LDAP and Active Directory adapter - #672
Conversation
Signed-off-by: XiaoSeS <87064762+XiaoSeS@users.noreply.github.com>
Signed-off-by: XiaoSeS <87064762+XiaoSeS@users.noreply.github.com>
|
Heads-up on a collision: #437 (@jangrui) and this PR both implement enterprise LDAP/AD login, and they'd conflict if both landed.
Both handle OpenLDAP This needs a maintainer to pick the lane before either merges, otherwise it's duplicated work plus a fork in the auth architecture. @dongjiang1989 — which direction do you want LDAP to take: the standalone #437, or folded into the federation core here? Whoever isn't chosen can likely contribute their edge-case handling (the explicit-bind endpoint in #437 is a nice touch) into the winning PR rather than being wasted. Not casting a vote on the direction — flagging so the two efforts converge instead of racing. |
|
Maintainer validation update:
Promotion from |
Summary
Identity and security
Verification completed
Remaining Draft gates
Closes #671
Refs #628, #260, #437
This PR targets big-main only. It must not be merged to main without explicit maintainer confirmation.