[checkmk-commits] [Checkmk/checkmk] ac4568: 17248 FIX ldap: you can't create a new connection ...

Branch: refs/heads/master Home: https://github.com/Checkmk/checkmk Commit: ac4568a2d637a0e217a84df5e636fce44a81b0e4 https://github.com/Checkmk/checkmk/commit/ac4568a2d637a0e217a84df5e636fce44… Author: Gav <gavin.mcguigan(a)checkmk.com> Date: 2024-09-17 (Tue, 17 Sep 2024) Changed paths: A .werks/17248.md M cmk/gui/fields/custom_fields.py M cmk/gui/openapi/endpoints/ldap_connection/__init__.py M cmk/gui/openapi/endpoints/ldap_connection/internal_to_restapi_interface.py M cmk/gui/openapi/endpoints/ldap_connection/request_schemas.py M tests/unit/cmk/gui/openapi/test_ldap_connection.py Log Message: ----------- 17248 FIX ldap: you can't create a new connection with an existing suffix Via the REST-API is was previously possible to create more than one ldap connection that had the same suffix. This was incorrect and doesn't match the behaviour of the UI. This werk addresses this issue by no longer allowing the same suffix on more than one ldap connection. CMK-19054 Change-Id: If7d89ae03e34007cb2322d9df04f206373f0b257 Commit: c86452cb7d1bdd7f3fdc85d8219fdeb18d09a376 https://github.com/Checkmk/checkmk/commit/c86452cb7d1bdd7f3fdc85d8219fdeb18… Author: Gav <gavin.mcguigan(a)checkmk.com> Date: 2024-09-17 (Tue, 17 Sep 2024) Changed paths: M cmk/gui/openapi/endpoints/ldap_connection/internal_to_restapi_interface.py M tests/unit/cmk/gui/openapi/test_ldap_connection.py Log Message: ----------- ldap: update ldap suffixes when creating, editing or deleting ldap connections The LDAPUserConnector class holds a dict[suffix, ldap_id]. On creating, deleting or editing LDAP connections via the REST-API, this wasn't taken into consideration. We now update these suffixes any time there are any modifications to the user ldap connections via the REST-API. CMK-19054 Change-Id: Iebc0049c18124f3cbd2257ec8a5a5c8ec8036340 Compare: https://github.com/Checkmk/checkmk/compare/9e1cea48c680...c86452cb7d1b To unsubscribe from these emails, change your notification settings at https://github.com/Checkmk/checkmk/settings/notifications