[GH-ISSUE #431] SMTP ENABLE_PASSWORD_RESET #163

New issue

Closed

opened 2026-02-27 08:15:37 +03:00 by kerem · 1 comment

kerem commented

2026-02-27 08:15:37 +03:00

Owner

Originally created by @zer0ish on GitHub (Feb 1, 2023).
Original GitHub issue: https://github.com/lldap/lldap/issues/431

I just changed my settings so that I can modify the SMTP info using the variable option in Unraid docker container instead of going to the config file.
I re-commented them in the config.
Set the variables in the container.
I wanted to test this variable to see if it works and noticed that if this one is set to False, the reset link still goes through.
Container Variable: LLDAP_SMTP_OPTIONS__ENABLE_PASSWORD_RESET

Shouldn't this either remove the reset option from the GUI or have a error message and not send a reset link?

Also, is it needed to have the double underscores for these? The config file shows the example with double underscore. But wanted to make sure it wasn't a typo.

Originally created by @zer0ish on GitHub (Feb 1, 2023). Original GitHub issue: https://github.com/lldap/lldap/issues/431 I just changed my settings so that I can modify the SMTP info using the variable option in Unraid docker container instead of going to the config file. I re-commented them in the config. Set the variables in the container. I wanted to test this variable to see if it works and noticed that if this one is set to False, the reset link still goes through. Container Variable: LLDAP_SMTP_OPTIONS__ENABLE_PASSWORD_RESET Shouldn't this either remove the reset option from the GUI or have a error message and not send a reset link? Also, is it needed to have the double underscores for these? The config file shows the example with double underscore. But wanted to make sure it wasn't a typo.

kerem

2026-02-27 08:15:37 +03:00

closed this issue
added the
enhancement

good first issue

frontend

backend
labels

kerem commented

2026-02-27 08:15:38 +03:00

Author

Owner

@nitnelave commented on GitHub (Feb 1, 2023):

Hmm, it seems we don't actually use that config value anywhere. We should probably add an endpoint on the server to check whether it's supported (like /server_info returning {password_reset_supported: true}), check that on page load in the client and change the UI in case it's not supported.

Good callout!

It shouldn't be too hard, if someone wants to pick this up.

@nitnelave commented on GitHub (Feb 1, 2023): Hmm, it seems we don't _actually_ use that config value anywhere. We should probably add an endpoint on the server to check whether it's supported (like `/server_info` returning `{password_reset_supported: true}`), check that on page load in the client and change the UI in case it's not supported. Good callout! It shouldn't be too hard, if someone wants to pick this up.

kerem referenced this issue

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

[PR #163] [MERGED] build(deps): bump docker/login-action from 1 to 2 #583