-
Notifications
You must be signed in to change notification settings - Fork 40
extended logging of aws authorization error #347
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, minor changes, would also ask @yevgenypats to review the message if its clearer now
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. One question :
What about if it’s not in an environment variable but just in ~/.aws/credentials
?
there is an
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
fixes #245