Skip to content

Commit

Permalink
PR feedback
Browse files Browse the repository at this point in the history
  • Loading branch information
woutslakhorst committed Nov 10, 2023
1 parent 2fae722 commit 0977af2
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions auth/client/iam/client.go
Original file line number Diff line number Diff line change
Expand Up @@ -155,9 +155,9 @@ func (hb HTTPClient) AccessToken(ctx context.Context, tokenEndpoint string, vp v
}
if err = core.TestResponseCode(http.StatusOK, response); err != nil {
// check for oauth error
if innerErr := core.TestResponseCode(http.StatusBadRequest, response); innerErr == nil {
if innerErr := core.TestResponseCode(http.StatusBadRequest, response); innerErr != nil {
// a non oauth error, the response body could contain a lot of stuff. We'll log and return the entire error
log.Logger().Errorf("authorization server token endpoint returned non oauth error (statusCode=%d)", response.StatusCode)
log.Logger().Debugf("authorization server token endpoint returned non oauth error (statusCode=%d)", response.StatusCode)
}

return token, err
Expand Down
2 changes: 1 addition & 1 deletion auth/services/oauth/relying_party.go
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@ type relyingParty struct {
wallet holder.Wallet
}

// NewRelyingParty returns an implementation of OAuthRelyingParty
// NewRelyingParty returns an implementation of RelyingParty
func NewRelyingParty(
didResolver resolver.DIDResolver, serviceResolver didman.CompoundServiceResolver, privateKeyStore nutsCrypto.KeyStore,
wallet holder.Wallet, httpClientTimeout time.Duration, httpClientTLS *tls.Config, strictMode bool) RelyingParty {
Expand Down

0 comments on commit 0977af2

Please sign in to comment.