Skip to content

Commit

Permalink
avoid a panic on failed creds polling (#7257)
Browse files Browse the repository at this point in the history
Backport for issue #7238
  • Loading branch information
jefferai authored Aug 5, 2019
1 parent 33d368e commit f26452d
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion command/agent/auth/alicloud/alicloud.go
Original file line number Diff line number Diff line change
Expand Up @@ -207,7 +207,7 @@ func (a *alicloudMethod) pollForCreds(credProvider providers.Provider, frequency
return
case <-ticker.C:
if err := a.checkCreds(credProvider); err != nil {
a.logger.Warn("unable to retrieve current creds, retaining last creds", err)
a.logger.Warn("unable to retrieve current creds, retaining last creds", "error", err)
}
}
}
Expand Down
2 changes: 1 addition & 1 deletion command/agent/auth/aws/aws.go
Original file line number Diff line number Diff line change
Expand Up @@ -279,7 +279,7 @@ func (a *awsMethod) pollForCreds(accessKey, secretKey, sessionToken string, freq
return
case <-ticker.C:
if err := a.checkCreds(accessKey, secretKey, sessionToken); err != nil {
a.logger.Warn("unable to retrieve current creds, retaining last creds", err)
a.logger.Warn("unable to retrieve current creds, retaining last creds", "error", err)
}
}
}
Expand Down

0 comments on commit f26452d

Please # to comment.