Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: expand env forcibly and double-check if file exists, make proper Go module #2

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

JerusJ
Copy link

@JerusJ JerusJ commented Oct 14, 2024

Just a couple changes:

  • Set project go.mod to match project repo (so it can be go install'd)
  • Fix shell expansion in cli args, also check if missing
  • Update AWS SDK
  • Change errors to just log.Fatal so we don't have to return/format

NOTE: the error:

Error getting identity: operation error STS: GetCallerIdentity, get identity: get credentials: failed to refresh cached credentials, no EC2 IMDS role found, operation error ec2imds: GetMetadata, request canceled, context deadline exceeded

Is only called if all credential providers are called but cannot be found. Since the config is loaded via: config.WithSharedCredentialsFiles([]string{sessionArgs.credFile}), if that is not correct/exists, then this red-herring error message is displayed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants