[PR #246] [MERGED] Set admin email from config #642

New issue

Closed

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

kerem commented

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

Owner

📋 Pull Request Information

Original PR: https://github.com/lldap/lldap/pull/246
Author: @ghost
Created: 7/13/2022
Status: ✅ Merged
Merged: 7/13/2022
Merged by: @nitnelave

Base: main ← Head: fm.admin_email

📝 Commits (1)

0f9b0af allow admin email to be set via config

📊 Changes

3 files changed (+8 additions, -0 deletions)

View changed files

📝 lldap_config.docker_template.toml (+5 -0)
📝 server/src/infra/configuration.rs (+2 -0)
📝 server/src/main.rs (+1 -0)

📄 Description

Allows folks to set ldap_user_email alongside ldap_user_dn and ldap_user_pass via the configuration file / environment variables. I need to have my admin account provisioned with an email at creation, so this ends up being pretty useful :)

I'm having issues with building a Docker image / the linker (on Debian). but I don't think due to my changes? Please let me know if I've forgotten anything!

_{🔄 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/246 **Author:** [@ghost](https://github.com/ghost) **Created:** 7/13/2022 **Status:** ✅ Merged **Merged:** 7/13/2022 **Merged by:** [@nitnelave](https://github.com/nitnelave) **Base:** `main` ← **Head:** `fm.admin_email` --- ### 📝 Commits (1) - [`0f9b0af`](https://github.com/lldap/lldap/commit/0f9b0af1c189e47ca3bd8d0cafea7199581fd796) allow admin email to be set via config ### 📊 Changes **3 files changed** (+8 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `lldap_config.docker_template.toml` (+5 -0) 📝 `server/src/infra/configuration.rs` (+2 -0) 📝 `server/src/main.rs` (+1 -0) </details> ### 📄 Description Allows folks to set `ldap_user_email` alongside `ldap_user_dn` and `ldap_user_pass` via the configuration file / environment variables. I need to have my admin account provisioned with an email at creation, so this ends up being pretty useful :) I'm having issues with building a Docker image / the linker (on Debian). but I don't think due to my changes? Please let me know if I've forgotten anything! --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>