Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[stable29] fix(user_ldap): Do not map groups we do not know if they match filter#50479

Closed
backportbot[bot] wants to merge 1 commit intostable29from
backport/45364/stable29
Closed

[stable29] fix(user_ldap): Do not map groups we do not know if they match filter#50479
backportbot[bot] wants to merge 1 commit intostable29from
backport/45364/stable29

Conversation

@backportbot
Copy link

@backportbot backportbot bot commented Jan 27, 2025

Backport of #45364

Warning, This backport's changes differ from the original and might be incomplete ⚠️

Todo

  • Review and resolve any conflicts
  • Amend HEAD commit to remove the line stating to skip CI

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

When nesting is enabled, filterValidGroups is supposed to check for each

groups if it actually exist, because it may not be visible to

Nextcloud. So in this codepath we disable automapping of groups.

Signed-off-by: Côme Chilliet <[email protected]>

[skip ci]
@backportbot backportbot bot added 3. to review Waiting for reviews feature: ldap labels Jan 27, 2025
@backportbot backportbot bot added this to the Nextcloud 29.0.12 milestone Jan 27, 2025
@come-nc
Copy link
Contributor

come-nc commented Jan 28, 2025

Will be replaced by a backport of #50480

@come-nc come-nc closed this Jan 28, 2025
@susnux susnux deleted the backport/45364/stable29 branch November 1, 2025 11:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant