Skip to content

Conversation

@dragonchaser
Copy link
Contributor

@dragonchaser dragonchaser commented Oct 28, 2025

This PR makes the user cache in the identity package of the graph service tenant aware
fixes #1462

@dragonchaser dragonchaser force-pushed the make-user-cache-tenant-aware branch 2 times, most recently from 5d721b4 to e79309f Compare November 5, 2025 16:13
@dragonchaser dragonchaser changed the title make user cache tenant aweare make user cache tenant aware Nov 5, 2025
Signed-off-by: Christian Richter <c.richter@opencloud.eu>
Signed-off-by: Christian Richter <c.richter@opencloud.eu>
Signed-off-by: Christian Richter <c.richter@opencloud.eu>
Signed-off-by: Christian Richter <c.richter@opencloud.eu>
Signed-off-by: Christian Richter <c.richter@opencloud.eu>
Signed-off-by: Christian Richter <c.richter@opencloud.eu>
Signed-off-by: Christian Richter <c.richter@opencloud.eu>
Signed-off-by: Christian Richter <c.richter@opencloud.eu>
@dragonchaser dragonchaser force-pushed the make-user-cache-tenant-aware branch from 7ac2a8a to 7fe511d Compare November 6, 2025 17:02
@dragonchaser dragonchaser marked this pull request as ready for review November 6, 2025 17:04
Signed-off-by: Christian Richter <c.richter@opencloud.eu>
@dragonchaser dragonchaser requested a review from rhafer November 11, 2025 16:17
Copy link
Contributor

@rhafer rhafer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks fine so far. Just some thoughts regarding the remaining TODO comments and a small simplification.

Signed-off-by: Christian Richter <c.richter@opencloud.eu>
Signed-off-by: Christian Richter <c.richter@opencloud.eu>
Signed-off-by: Christian Richter <c.richter@opencloud.eu>
@dragonchaser dragonchaser merged commit acc38eb into opencloud-eu:main Nov 13, 2025
58 checks passed
@openclouders openclouders mentioned this pull request Nov 13, 2025
1 task
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.

User cache in graph service is not multi-tenant safe

2 participants