[PR #149] [MERGED] Accept "uid" in qualified name #571

New issue

Closed

opened 2026-02-27 08:18:02 +03:00 by kerem · 0 comments

kerem commented

2026-02-27 08:18:02 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/lldap/lldap/pull/149
Author: @nitnelave
Created: 4/29/2022
Status: ✅ Merged
Merged: 4/29/2022
Merged by: @nitnelave

Base: main ← Head: uid

📝 Commits (2)

7d27ec7 ldap: accept "uid" or "cn" as username
36478bc ldap: return actual "cn" value instead of "uid" in LDAP messages

📊 Changes

7 files changed (+51 additions, -33 deletions)

View changed files

📝 example_configs/authelia_config.yml (+1 -1)
📝 example_configs/bookstack.env.example (+2 -2)
📝 example_configs/dolibarr.md (+12 -13)
📝 example_configs/jellyfin.md (+13 -1)
📝 example_configs/jitsi_meet.conf (+1 -1)
📝 example_configs/keycloak.md (+1 -1)
📝 server/src/infra/ldap_handler.rs (+21 -14)

📄 Description

This also changes the recommendation to use "uid" instead of "cn", since that's the meaning of the field. Using the actual value of "cn" will not actually work.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/lldap/lldap/pull/149 **Author:** [@nitnelave](https://github.com/nitnelave) **Created:** 4/29/2022 **Status:** ✅ Merged **Merged:** 4/29/2022 **Merged by:** [@nitnelave](https://github.com/nitnelave) **Base:** `main` ← **Head:** `uid` --- ### 📝 Commits (2) - [`7d27ec7`](https://github.com/lldap/lldap/commit/7d27ec7b54fb2a58636f3bda9218665d6def83d3) ldap: accept "uid" or "cn" as username - [`36478bc`](https://github.com/lldap/lldap/commit/36478bc86ca500915c8b33df3cc2a7448628e2b9) ldap: return actual "cn" value instead of "uid" in LDAP messages ### 📊 Changes **7 files changed** (+51 additions, -33 deletions) <details> <summary>View changed files</summary> 📝 `example_configs/authelia_config.yml` (+1 -1) 📝 `example_configs/bookstack.env.example` (+2 -2) 📝 `example_configs/dolibarr.md` (+12 -13) 📝 `example_configs/jellyfin.md` (+13 -1) 📝 `example_configs/jitsi_meet.conf` (+1 -1) 📝 `example_configs/keycloak.md` (+1 -1) 📝 `server/src/infra/ldap_handler.rs` (+21 -14) </details> ### 📄 Description This also changes the recommendation to use "uid" instead of "cn", since that's the meaning of the field. Using the actual value of "cn" will not actually work. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>