[PR #418] [MERGED] Improve support for MySQL/Postgres #728

New issue

Closed

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

kerem commented

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

Owner

📋 Pull Request Information

Original PR: https://github.com/lldap/lldap/pull/418
Author: @nitnelave
Created: 1/13/2023
Status: ✅ Merged
Merged: 1/13/2023
Merged by: @nitnelave

Base: main ← Head: mysql

📝 Commits (3)

312230e db: Change the version number from u8 to i16
70e8a94 db: Change the DB storage type to NaiveDateTime
78a9bd6 clippy: fix warning

📊 Changes

16 files changed (+65 additions, -45 deletions)

View changed files

📝 server/src/domain/handler.rs (+8 -2)
📝 server/src/domain/ldap/user.rs (+5 -1)
📝 server/src/domain/model/groups.rs (+1 -1)
📝 server/src/domain/model/jwt_refresh_storage.rs (+1 -1)
📝 server/src/domain/model/jwt_storage.rs (+1 -1)
📝 server/src/domain/model/password_reset_tokens.rs (+1 -1)
📝 server/src/domain/model/users.rs (+1 -1)
📝 server/src/domain/sql_group_backend_handler.rs (+1 -1)
📝 server/src/domain/sql_migrations.rs (+4 -3)
📝 server/src/domain/sql_opaque_handler.rs (+1 -1)
📝 server/src/domain/sql_tables.rs (+4 -4)
📝 server/src/domain/sql_user_backend_handler.rs (+1 -1)
📝 server/src/domain/types.rs (+14 -9)
📝 server/src/infra/graphql/query.rs (+6 -5)
📝 server/src/infra/ldap_handler.rs (+14 -11)
📝 server/src/infra/sql_backend_handler.rs (+2 -2)

📄 Description

With these 2 commits, the integration with mysql/postgres is working.

It was only tested by hand, I didn't set up an automated test.

It's completely backwards-compatible with SQlite.

Fixes #87

_{🔄 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/418 **Author:** [@nitnelave](https://github.com/nitnelave) **Created:** 1/13/2023 **Status:** ✅ Merged **Merged:** 1/13/2023 **Merged by:** [@nitnelave](https://github.com/nitnelave) **Base:** `main` ← **Head:** `mysql` --- ### 📝 Commits (3) - [`312230e`](https://github.com/lldap/lldap/commit/312230e45f492c5af3a35dad198b40356d0274f7) db: Change the version number from u8 to i16 - [`70e8a94`](https://github.com/lldap/lldap/commit/70e8a94328e9ba131de785676cd27c660e405df5) db: Change the DB storage type to NaiveDateTime - [`78a9bd6`](https://github.com/lldap/lldap/commit/78a9bd638db724f981c654fcd3e047c8b169b8ed) clippy: fix warning ### 📊 Changes **16 files changed** (+65 additions, -45 deletions) <details> <summary>View changed files</summary> 📝 `server/src/domain/handler.rs` (+8 -2) 📝 `server/src/domain/ldap/user.rs` (+5 -1) 📝 `server/src/domain/model/groups.rs` (+1 -1) 📝 `server/src/domain/model/jwt_refresh_storage.rs` (+1 -1) 📝 `server/src/domain/model/jwt_storage.rs` (+1 -1) 📝 `server/src/domain/model/password_reset_tokens.rs` (+1 -1) 📝 `server/src/domain/model/users.rs` (+1 -1) 📝 `server/src/domain/sql_group_backend_handler.rs` (+1 -1) 📝 `server/src/domain/sql_migrations.rs` (+4 -3) 📝 `server/src/domain/sql_opaque_handler.rs` (+1 -1) 📝 `server/src/domain/sql_tables.rs` (+4 -4) 📝 `server/src/domain/sql_user_backend_handler.rs` (+1 -1) 📝 `server/src/domain/types.rs` (+14 -9) 📝 `server/src/infra/graphql/query.rs` (+6 -5) 📝 `server/src/infra/ldap_handler.rs` (+14 -11) 📝 `server/src/infra/sql_backend_handler.rs` (+2 -2) </details> ### 📄 Description With these 2 commits, the integration with mysql/postgres is working. It was only tested by hand, I didn't set up an automated test. It's completely backwards-compatible with SQlite. Fixes #87 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>