Skip to content

Fix degradation in 3130bfe#15

Merged
tsub merged 1 commit intomasterfrom
fixup-pr13
Aug 23, 2018
Merged

Fix degradation in 3130bfe#15
tsub merged 1 commit intomasterfrom
fixup-pr13

Conversation

@tsub
Copy link
Copy Markdown
Owner

@tsub tsub commented Aug 23, 2018

Why?

$AWS_PROFILE is not worked.

$ AWS_PROFILE=myprofile aws configure get region
ap-northeast-1
$ AWS_PROFILE=myprofile s3-edit edit s3://tsub-sandbox/test.txt
MissingRegion: could not find region configuration

$ AWS_PROFILE=myprofile AWS_REGION=ap-northeast-1 s3-edit edit s3://tsub-sandbox/test.txt
NoCredentialProviders: no valid providers in chain. Deprecated.
        For verbose messaging see aws.Config.CredentialsChainVerboseErrors

How?

Do not specify default profile when --profile is not set.

$ AWS_PROFILE=myprofile go run main.go edit s3://tsub-sandbox/test.txt # is working

$ AWS_PROFILE=myprofile aws configure get region
ap-northeast-1

`$AWS_PROFILE` is not used
@tsub tsub self-assigned this Aug 23, 2018
@tsub tsub merged commit bbca85c into master Aug 23, 2018
@tsub tsub deleted the fixup-pr13 branch August 23, 2018 14:04
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.

1 participant