diff --git a/internal/service/oidc_service.go b/internal/service/oidc_service.go index 92216451..b263cc66 100644 --- a/internal/service/oidc_service.go +++ b/internal/service/oidc_service.go @@ -296,6 +296,11 @@ func (service *OIDCService) ValidateAuthorizeParams(req AuthorizeRequest) error if !ok { return errors.New("access_denied") } + + // Redirect URI to verify that it's trusted + if !slices.Contains(client.TrustedRedirectURIs, req.RedirectURI) { + return errors.New("invalid_request_uri") + } // Scopes scopes := strings.Split(req.Scope, " ") @@ -318,11 +323,6 @@ func (service *OIDCService) ValidateAuthorizeParams(req AuthorizeRequest) error return errors.New("unsupported_response_type") } - // Redirect URI - if !slices.Contains(client.TrustedRedirectURIs, req.RedirectURI) { - return errors.New("invalid_request_uri") - } - // PKCE code challenge method if set if req.CodeChallenge != "" && req.CodeChallengeMethod != "" { if req.CodeChallengeMethod != "S256" && req.CodeChallengeMethod != "plain" {