[PR #626] [MERGED] ldap: Add support for modifying the password with a modify operation #855

New issue

Closed

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

kerem commented

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

Owner

📋 Pull Request Information

Original PR: https://github.com/lldap/lldap/pull/626
Author: @nitnelave
Created: 7/29/2023
Status: ✅ Merged
Merged: 7/29/2023
Merged by: @nitnelave

Base: main ← Head: password_modify

📝 Commits (1)

79b6c01 ldap: Add support for modifying the password with a modify operation

📊 Changes

8 files changed (+183 additions, -17 deletions)

View changed files

📝 app/src/components/change_password.rs (+5 -3)
📝 app/src/components/create_user.rs (+4 -1)
📝 app/src/components/reset_password_step2.rs (+1 -1)
📝 auth/src/opaque.rs (+2 -2)
📝 server/src/domain/sql_backend_handler.rs (+1 -1)
📝 server/src/domain/sql_opaque_handler.rs (+1 -1)
📝 server/src/infra/ldap_handler.rs (+168 -7)
📝 set-password/src/main.rs (+1 -1)

📄 Description

Fixes #620

Note that in verbose logging mode, the password will be written in clear in the logs. That was the original motivation for only supporting the extended operation.

_{🔄 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/626 **Author:** [@nitnelave](https://github.com/nitnelave) **Created:** 7/29/2023 **Status:** ✅ Merged **Merged:** 7/29/2023 **Merged by:** [@nitnelave](https://github.com/nitnelave) **Base:** `main` ← **Head:** `password_modify` --- ### 📝 Commits (1) - [`79b6c01`](https://github.com/lldap/lldap/commit/79b6c0147d535fdda90378f28eecb0d8dfb337b7) ldap: Add support for modifying the password with a modify operation ### 📊 Changes **8 files changed** (+183 additions, -17 deletions) <details> <summary>View changed files</summary> 📝 `app/src/components/change_password.rs` (+5 -3) 📝 `app/src/components/create_user.rs` (+4 -1) 📝 `app/src/components/reset_password_step2.rs` (+1 -1) 📝 `auth/src/opaque.rs` (+2 -2) 📝 `server/src/domain/sql_backend_handler.rs` (+1 -1) 📝 `server/src/domain/sql_opaque_handler.rs` (+1 -1) 📝 `server/src/infra/ldap_handler.rs` (+168 -7) 📝 `set-password/src/main.rs` (+1 -1) </details> ### 📄 Description Fixes #620 Note that in verbose logging mode, the password will be written in clear in the logs. That was the original motivation for only supporting the extended operation. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>