Skip to content

fix: allow to not delete memberships during synchro if the information is not present in LDAP - EXO-87414 - #136

Merged
rdenarie merged 1 commit into
developfrom
backport/task-87414
Jun 15, 2026
Merged

rdenarie merged 1 commit into
developfrom
backport/task-87414

Conversation

@rdenarie

Copy link
Copy Markdown
Member

Sometimes in LDAP, the user membership are not store in 2-directions (group to user with member attribute AND user to group with memberOf attribute) but only in one direction (group to user OR user to group). In this case, when synchronizing it, membership can be removed in exo : If it is stored in group-to-user direction, when you synchronize the user, membership are not seen and then removed in exo. If it is stored in user-to-group direction, when you synchronize the group, membership are not seen and then removed in exo.

This commit add 2 properties to be able to not delete the membership is not present in user or group object exo.idm.externalStore.user.memberships.update.delete exo.idm.externalStore.group.memberships.update.delete

In addition, the commit ensure to correctly use this attribute to ignore membership which are not present it the attribute is true

…n is not present in LDAP - EXO-87414

Sometimes in LDAP, the user membership are not store in 2-directions (group to user with member attribute AND user to group with memberOf attribute) but only in one direction (group to user OR user to group).
In this case, when synchronizing it, membership can be removed in exo :
If it is stored in group-to-user direction, when you synchronize the user, membership are not seen and then removed in exo.
If it is stored in user-to-group direction, when you synchronize the group, membership are not seen and then removed in exo.

This commit add 2 properties to be able to not delete the membership is not present in user or group object
exo.idm.externalStore.user.memberships.update.delete
exo.idm.externalStore.group.memberships.update.delete

In addition, the commit ensure to correctly use this attribute to ignore membership which are not present it the attribute is true
@rdenarie
rdenarie requested a review from Jihed525 June 15, 2026 07:20
@rdenarie
rdenarie enabled auto-merge (rebase) June 15, 2026 07:20
@sonarqubecloud

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
28.6% Coverage on New Code (required ≥ 60%)

See analysis details on SonarQube Cloud

@rdenarie
rdenarie merged commit 227db7a into develop Jun 15, 2026
8 of 9 checks passed
@rdenarie
rdenarie deleted the backport/task-87414 branch June 15, 2026 07:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants