Skip to content

[stable32] perf: Don't fetch full group object in AuthorizedGroupMapper - #61850

Merged
CarlSchwan merged 1 commit into
stable32from
backport/61838/stable32
Aug 10, 2026
Merged

[stable32] perf: Don't fetch full group object in AuthorizedGroupMapper#61850
CarlSchwan merged 1 commit into
stable32from
backport/61838/stable32

Conversation

@backportbot

@backportbotbackportbotBot commented Jul 6, 2026

Copy link
Copy Markdown

Backport of #61838

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

Todo

  • Review and resolve any conflicts
  • Review and verify the backported changes
  • 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.

@backportbotbackportbotBot added this to the Nextcloud 32.0.13 milestone Jul 6, 2026
@CarlSchwan
CarlSchwanforce-pushed the backport/61838/stable32 branch from 244782e to 883c21cCompareJuly 6, 2026 15:05
@CarlSchwan
CarlSchwan marked this pull request as ready for review July 6, 2026 15:06
@CarlSchwan
CarlSchwan requested a review from a team as a code ownerJuly 6, 2026 15:06
@AndyScherzinger
AndyScherzingerforce-pushed the backport/61838/stable32 branch from 883c21c to 2194deeCompareJuly 8, 2026 19:37
@DerDreschner
DerDreschnerforce-pushed the backport/61838/stable32 branch from 2194dee to 949e975CompareJuly 12, 2026 22:49
@nextcloud-botnextcloud-bot mentioned this pull request Jul 14, 2026
@AndyScherzinger
AndyScherzingerforce-pushed the backport/61838/stable32 branch from 949e975 to 05b9ebaCompareJuly 15, 2026 22:54
This is very expensive when multiple group backends are enabled as we
don't cache which groups is included in which backend unlike in the User
Manager.
Signed-off-by: Carl Schwan <carlschwan@kde.org>
@AndyScherzinger
AndyScherzingerforce-pushed the backport/61838/stable32 branch from 05b9eba to 98e67deCompareJuly 15, 2026 23:11
@nextcloud-botnextcloud-bot mentioned this pull request Aug 10, 2026
@CarlSchwan
CarlSchwan merged commit 09dc693 into stable32Aug 10, 2026
214 of 226 checks passed
@CarlSchwan
CarlSchwan deleted the backport/61838/stable32 branch August 10, 2026 09:29
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewsperformance 🚀

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@CarlSchwan@provokateurin@AndyScherzinger