From 869f005136257ed0ac9ff34606407402b07c2756 Mon Sep 17 00:00:00 2001 From: binwiederhier Date: Mon, 1 Jun 2026 21:15:43 -0400 Subject: [PATCH] Review --- user/manager.go | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/user/manager.go b/user/manager.go index bae32593..b036df73 100644 --- a/user/manager.go +++ b/user/manager.go @@ -298,11 +298,8 @@ func (a *Manager) ChangeRole(username string, role Role) error { if err != nil { return err } - // Promotion to admin clears user_access rows for this user AND rows owned - // by this user (Everyone rows from their reservations). Other role changes - // are no-ops for the cache but reloading the two affected slices is cheap - // and keeps the code path uniform. - return a.maybeReloadAccessCache(username, Everyone) + // Full cache reload: Role changes are extremely rare. + return a.maybeReloadAccessCache() } // changeRoleTx changes a user's role