Fix modify samaccountname

[Naksen]

No description provided.

[Copilot]

Pull request overview

This PR updates the LDAP modify flow to correctly rename Kerberos principals when sAMAccountName / userPrincipalName changes, by introducing a dedicated rename_princ operation across the Kerberos client/server stubs and adjusting tests accordingly.

Changes:

• Switch LDAP modify logic from kadmin.modify_princ(...) to kadmin.rename_princ(...) for user/computer sAMAccountName-related renames.
• Add rename_princ to the Kerberos kadmin abstraction (base/stub/client) and implement a new /principal/rename endpoint in the Kerberos config server.
• Update API tests and test kadmin mocks to assert rename_princ usage.

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
app/ldap_protocol/ldap_requests/modify.py	Uses rename_princ for principal renames and adjusts the RDN modification guard.
app/ldap_protocol/kerberos/base.py	Extends AbstractKadmin with rename_princ.
app/ldap_protocol/kerberos/client.py	Fixes modify endpoint payload/path; adds rename_princ client call.
app/ldap_protocol/kerberos/stub.py	Adds stubbed rename_princ.
.kerberos/config_server.py	Adds manager method + /principal/rename route to support principal renaming.
tests/conftest.py	Extends the mocked kadmin with rename_princ.
tests/test_api/test_main/test_router/test_modify.py	Updates assertions to expect rename_princ calls.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.