From e4c45c4f9819fa2f27cf8a824c44f0ea2a9c5553 Mon Sep 17 00:00:00 2001 From: Karl Persson Date: Tue, 9 Aug 2022 15:52:14 +0200 Subject: [PATCH] RBAC: Remove sort and unique values when fetching permissions (#53461) * RBAC: remove sort and unique values for fetching permissions --- pkg/services/accesscontrol/database/database.go | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/pkg/services/accesscontrol/database/database.go b/pkg/services/accesscontrol/database/database.go index 8b704ddd7bb..eac6ba968f8 100644 --- a/pkg/services/accesscontrol/database/database.go +++ b/pkg/services/accesscontrol/database/database.go @@ -26,8 +26,7 @@ func (s *AccessControlStore) GetUserPermissions(ctx context.Context, query acces err := s.sql.WithDbSession(ctx, func(sess *sqlstore.DBSession) error { filter, params := userRolesFilter(query.OrgID, query.UserID, query.Roles) - // TODO: optimize this - q := `SELECT DISTINCT + q := `SELECT permission.action, permission.scope FROM permission @@ -45,10 +44,6 @@ func (s *AccessControlStore) GetUserPermissions(ctx context.Context, query acces } } - q += ` - ORDER BY permission.scope - ` - if err := sess.SQL(q, params...).Find(&result); err != nil { return err }