[PR #129] [MERGED] domain: introduce UserId to make uid case insensitive #557

New issue

Closed

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

kerem commented

2026-02-27 08:17:58 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/lldap/lldap/pull/129
Author: @nitnelave
Created: 3/26/2022
Status: ✅ Merged
Merged: 3/26/2022
Merged by: @nitnelave

Base: main ← Head: user_id

📝 Commits (1)

3e675be domain: introduce UserId to make uid case insensitive

📊 Changes

13 files changed (+299 additions, -181 deletions)

View changed files

📝 server/src/domain/error.rs (+1 -1)
📝 server/src/domain/handler.rs (+49 -17)
📝 server/src/domain/opaque_handler.rs (+3 -3)
📝 server/src/domain/sql_backend_handler.rs (+82 -50)
📝 server/src/domain/sql_opaque_handler.rs (+25 -14)
📝 server/src/domain/sql_tables.rs (+38 -1)
📝 server/src/infra/auth_service.rs (+10 -10)
📝 server/src/infra/configuration.rs (+6 -3)
📝 server/src/infra/graphql/mutation.rs (+8 -7)
📝 server/src/infra/graphql/query.rs (+12 -9)
📝 server/src/infra/ldap_handler.rs (+43 -44)
📝 server/src/infra/sql_backend_handler.rs (+6 -6)
📝 server/src/infra/tcp_backend_handler.rs (+16 -16)

📄 Description

Note that if there was a non-lowercase user already in the DB, it cannot
be found again. To fix this, run in the DB:

sqlite> UPDATE users SET user_id = LOWER(user_id);

Fixes #125

_{🔄 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/129 **Author:** [@nitnelave](https://github.com/nitnelave) **Created:** 3/26/2022 **Status:** ✅ Merged **Merged:** 3/26/2022 **Merged by:** [@nitnelave](https://github.com/nitnelave) **Base:** `main` ← **Head:** `user_id` --- ### 📝 Commits (1) - [`3e675be`](https://github.com/lldap/lldap/commit/3e675be838a9996580d04f68ffb113225f39ac1a) domain: introduce UserId to make uid case insensitive ### 📊 Changes **13 files changed** (+299 additions, -181 deletions) <details> <summary>View changed files</summary> 📝 `server/src/domain/error.rs` (+1 -1) 📝 `server/src/domain/handler.rs` (+49 -17) 📝 `server/src/domain/opaque_handler.rs` (+3 -3) 📝 `server/src/domain/sql_backend_handler.rs` (+82 -50) 📝 `server/src/domain/sql_opaque_handler.rs` (+25 -14) 📝 `server/src/domain/sql_tables.rs` (+38 -1) 📝 `server/src/infra/auth_service.rs` (+10 -10) 📝 `server/src/infra/configuration.rs` (+6 -3) 📝 `server/src/infra/graphql/mutation.rs` (+8 -7) 📝 `server/src/infra/graphql/query.rs` (+12 -9) 📝 `server/src/infra/ldap_handler.rs` (+43 -44) 📝 `server/src/infra/sql_backend_handler.rs` (+6 -6) 📝 `server/src/infra/tcp_backend_handler.rs` (+16 -16) </details> ### 📄 Description Note that if there was a non-lowercase user already in the DB, it cannot be found again. To fix this, run in the DB: sqlite> UPDATE users SET user_id = LOWER(user_id); Fixes #125 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>