Commit 1911520e by Seefs

fix: normalize generic oauth bearer token type

parent ed6ff0f2
...@@ -208,10 +208,7 @@ func (p *GenericOAuthProvider) GetUserInfo(ctx context.Context, token *OAuthToke ...@@ -208,10 +208,7 @@ func (p *GenericOAuthProvider) GetUserInfo(ctx context.Context, token *OAuthToke
} }
// Set authorization header // Set authorization header
tokenType := token.TokenType tokenType := normalizeAuthorizationTokenType(token.TokenType)
if tokenType == "" {
tokenType = "Bearer"
}
req.Header.Set("Authorization", fmt.Sprintf("%s %s", tokenType, token.AccessToken)) req.Header.Set("Authorization", fmt.Sprintf("%s %s", tokenType, token.AccessToken))
req.Header.Set("Accept", "application/json") req.Header.Set("Accept", "application/json")
...@@ -320,6 +317,14 @@ func (p *GenericOAuthProvider) GetProviderId() int { ...@@ -320,6 +317,14 @@ func (p *GenericOAuthProvider) GetProviderId() int {
return p.config.Id return p.config.Id
} }
func normalizeAuthorizationTokenType(tokenType string) string {
tokenType = strings.TrimSpace(tokenType)
if tokenType == "" || strings.EqualFold(tokenType, "Bearer") {
return "Bearer"
}
return tokenType
}
// IsGenericProvider returns true for generic providers // IsGenericProvider returns true for generic providers
func (p *GenericOAuthProvider) IsGenericProvider() bool { func (p *GenericOAuthProvider) IsGenericProvider() bool {
return true return true
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or sign in to comment