[PR #555] [MERGED] server: Improve the error message in case of duplicate emails #821

New issue

Closed

opened 2026-02-27 09:09:51 +03:00 by kerem · 0 comments

kerem commented

2026-02-27 09:09:51 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/lldap/lldap/pull/555
Author: @nitnelave
Created: 4/14/2023
Status: ✅ Merged
Merged: 4/14/2023
Merged by: @nitnelave

Base: main ← Head: migration

📝 Commits (2)

e1856e6 server: Improve the error message in case of duplicate emails
81cf28b server: fix incorrect logging

📊 Changes

3 files changed (+138 additions, -17 deletions)

View changed files

➕ docs/migration_guides/v0.5.md (+58 -0)
📝 server/src/domain/sql_migrations.rs (+72 -14)
📝 server/src/domain/sql_tables.rs (+8 -3)

📄 Description

Example error message, from the test:

2023-04-14T09:44:38.905177Z  INFO lldap::domain::sql_migrations: Upgrading DB schema from version 1
2023-04-14T09:44:38.905208Z  INFO lldap::domain::sql_migrations: Upgrading DB schema to version 2
2023-04-14T09:44:38.910783Z  INFO lldap::domain::sql_migrations: Upgrading DB schema to version 3
2023-04-14T09:44:38.924604Z  INFO lldap::domain::sql_migrations: Upgrading DB schema to version 4
2023-04-14T09:40:08.738013Z  WARN lldap::domain::sql_migrations: Found several users with the same email:
2023-04-14T09:40:08.738613Z  WARN lldap::domain::sql_migrations: Email: bob@bob.com
2023-04-14T09:40:08.738642Z  WARN lldap::domain::sql_migrations:     User: bob
2023-04-14T09:40:08.738659Z  WARN lldap::domain::sql_migrations:     User: bob2
2023-04-14T09:40:08.738703Z ERROR lldap::domain::sql_tables::tests: while enforcing unicity on emails (2 users have the same email).

See https://github.com/lldap/lldap/blob/main/docs/migration_guides/v0.5.md for details.

_{🔄 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/555 **Author:** [@nitnelave](https://github.com/nitnelave) **Created:** 4/14/2023 **Status:** ✅ Merged **Merged:** 4/14/2023 **Merged by:** [@nitnelave](https://github.com/nitnelave) **Base:** `main` ← **Head:** `migration` --- ### 📝 Commits (2) - [`e1856e6`](https://github.com/lldap/lldap/commit/e1856e61b5255af92d59cb41708315552d58450c) server: Improve the error message in case of duplicate emails - [`81cf28b`](https://github.com/lldap/lldap/commit/81cf28b9d863f72ca5f2c11a9e564119a78dd0d9) server: fix incorrect logging ### 📊 Changes **3 files changed** (+138 additions, -17 deletions) <details> <summary>View changed files</summary> ➕ `docs/migration_guides/v0.5.md` (+58 -0) 📝 `server/src/domain/sql_migrations.rs` (+72 -14) 📝 `server/src/domain/sql_tables.rs` (+8 -3) </details> ### 📄 Description Example error message, from the test: ``` 2023-04-14T09:44:38.905177Z INFO lldap::domain::sql_migrations: Upgrading DB schema from version 1 2023-04-14T09:44:38.905208Z INFO lldap::domain::sql_migrations: Upgrading DB schema to version 2 2023-04-14T09:44:38.910783Z INFO lldap::domain::sql_migrations: Upgrading DB schema to version 3 2023-04-14T09:44:38.924604Z INFO lldap::domain::sql_migrations: Upgrading DB schema to version 4 2023-04-14T09:40:08.738013Z WARN lldap::domain::sql_migrations: Found several users with the same email: 2023-04-14T09:40:08.738613Z WARN lldap::domain::sql_migrations: Email: bob@bob.com 2023-04-14T09:40:08.738642Z WARN lldap::domain::sql_migrations: User: bob 2023-04-14T09:40:08.738659Z WARN lldap::domain::sql_migrations: User: bob2 2023-04-14T09:40:08.738703Z ERROR lldap::domain::sql_tables::tests: while enforcing unicity on emails (2 users have the same email). See https://github.com/lldap/lldap/blob/main/docs/migration_guides/v0.5.md for details. ``` --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>