From 578f49a7f651f73d10e1722544c4cf224dd1447c Mon Sep 17 00:00:00 2001 From: "grafana-delivery-bot[bot]" <132647405+grafana-delivery-bot[bot]@users.noreply.github.com> Date: Mon, 13 Jan 2025 12:05:54 +0200 Subject: [PATCH] [v11.3.x] OAuth: Return SocialError on membership check failure (#98860) OAuth: Return SocialError on membership check failure (#98330) use socialerror (cherry picked from commit ddca09fb021637474c19b0dd28549a62880c66aa) Co-authored-by: Daniel-Davies <33356828+Daniel-Davies@users.noreply.github.com> --- pkg/login/social/connectors/generic_oauth.go | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/pkg/login/social/connectors/generic_oauth.go b/pkg/login/social/connectors/generic_oauth.go index 1e386d1d2e2..eb4a32f8381 100644 --- a/pkg/login/social/connectors/generic_oauth.go +++ b/pkg/login/social/connectors/generic_oauth.go @@ -4,7 +4,6 @@ import ( "bytes" "context" "encoding/json" - "errors" "fmt" "net/http" "net/mail" @@ -320,11 +319,11 @@ func (s *SocialGenericOAuth) UserInfo(ctx context.Context, client *http.Client, } if !s.isTeamMember(ctx, client) { - return nil, errors.New("user not a member of one of the required teams") + return nil, &SocialError{"User not a member of one of the required teams"} } if !s.isOrganizationMember(ctx, client) { - return nil, errors.New("user not a member of one of the required organizations") + return nil, &SocialError{"User not a member of one of the required organizations"} } if !s.isGroupMember(userInfo.Groups) {