[PR #692] [MERGED] Remove password length limitation in lldap_set_password tool #902

New issue

Closed

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

kerem commented

2026-02-27 09:10:11 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/lldap/lldap/pull/692
Author: @stuart938503
Created: 9/28/2023
Status: ✅ Merged
Merged: 9/28/2023
Merged by: @nitnelave

Base: main ← Head: patch-1

📝 Commits (3)

56d48bc Remove password length limitation
5f8cb70 Added minimum length back but with ignore flag
012e214 Updated with suggestions from @nitnelave

📊 Changes

1 file changed (+5 additions, -1 deletions)

View changed files

📝 set-password/src/main.rs (+5 -1)

📄 Description

The password length limitation is arbitrary and non-configurable, and there are legitimate requirements to set a password shorter than 8 characters. This change will make it simple to achieve without the need for a third party tool.

I would update the web UI code also, but am unfamiliar with Rust and the wider LLDAP project to confidently make the change and not break something. The change for this tool appears simple, however.

_{🔄 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/692 **Author:** [@stuart938503](https://github.com/stuart938503) **Created:** 9/28/2023 **Status:** ✅ Merged **Merged:** 9/28/2023 **Merged by:** [@nitnelave](https://github.com/nitnelave) **Base:** `main` ← **Head:** `patch-1` --- ### 📝 Commits (3) - [`56d48bc`](https://github.com/lldap/lldap/commit/56d48bcd94782d06fbab7d7961eed37b001c0d25) Remove password length limitation - [`5f8cb70`](https://github.com/lldap/lldap/commit/5f8cb70b9b4f95a8275b02893de228a3f44baf56) Added minimum length back but with ignore flag - [`012e214`](https://github.com/lldap/lldap/commit/012e214b4a116b7bfe2b1aa10ce3b2d4e3c02dfa) Updated with suggestions from @nitnelave ### 📊 Changes **1 file changed** (+5 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `set-password/src/main.rs` (+5 -1) </details> ### 📄 Description The password length limitation is arbitrary and non-configurable, and there are legitimate requirements to set a password shorter than 8 characters. This change will make it simple to achieve without the need for a third party tool. I would update the web UI code also, but am unfamiliar with Rust and the wider LLDAP project to confidently make the change and not break something. The change for this tool appears simple, however. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>