Skip to content

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

Open
rdenarie wants to merge 2 commits into
feature/maintenancefrom
fix/task-87414
Open

fix: allow to not delete memberships during synchro if the information is not present in LDAP - EXO-87414#135
rdenarie wants to merge 2 commits into
feature/maintenancefrom
fix/task-87414

Conversation

@rdenarie

@rdenarie rdenarie commented Jun 4, 2026

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

halaya and others added 2 commits June 3, 2026 23:17
…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 review from ahamdi, azayati and mkrout June 4, 2026 15:59
@rdenarie rdenarie enabled auto-merge (rebase) June 4, 2026 15:59
@sonarqubecloud

sonarqubecloud Bot commented Jun 4, 2026

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

@exo-swf exo-swf force-pushed the feature/maintenance branch from 9a5f97c to 8f526aa Compare June 4, 2026 23:18
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