Skip to content

Commit

Permalink
Merge pull request #2033 from gobitfly/test/AddingMoreDebugInfosForOA…
Browse files Browse the repository at this point in the history
…UTHWarning

(NOBIDS) Adding more debug infos for OAUTH warning
  • Loading branch information
recy21 authored Apr 3, 2023
2 parents d982435 + e0c16fa commit b1edd17
Showing 1 changed file with 8 additions and 1 deletion.
9 changes: 8 additions & 1 deletion utils/oauthutils.go
Original file line number Diff line number Diff line change
Expand Up @@ -123,7 +123,13 @@ func accessTokenGetClaims(tokenStringFull string, validate bool) (*CustomClaims,

if err != nil && validate {
if !strings.Contains(err.Error(), "token is expired") {
logger.Warnf("Error parsing jwt token: %v %v", err, token)
logger.WithFields(
logrus.Fields{
"error": err,
"token": token,
"tokenString": tokenString,
},
).Warn("Error parsing jwt token")
}

return nil, err
Expand Down Expand Up @@ -203,6 +209,7 @@ func GetAuthorizationClaims(r *http.Request) *CustomClaims {

claims, err := ValidateAccessTokenGetClaims(accessToken)
if err != nil {
logger.Warnf("ValidateAccessTokenGetClaims failed") // #REMOVE just for test purpose, can be removed after testing
return nil
}
return claims
Expand Down

0 comments on commit b1edd17

Please sign in to comment.