Skip to content

Allow default ASM credential chain#3

Open
yohamta0 wants to merge 2 commits into
wealdtech:masterfrom
curvegrid:asm-default-credential-chain
Open

Allow default ASM credential chain#3
yohamta0 wants to merge 2 commits into
wealdtech:masterfrom
curvegrid:asm-default-credential-chain

Conversation

@yohamta0

@yohamta0 yohamta0 commented Apr 7, 2026

Copy link
Copy Markdown

Summary

  • when asm:///... is used without explicit credentials, ASM currently falls back to credentials.NewEnvCredentials()
  • that provider only supports raw access keys from environment variables and bypasses the AWS SDK default credential chain
  • this change leaves credentials unset in that case so the AWS SDK can resolve them through its normal default chain
  • explicit URL credentials and configured static credentials are unchanged

Testing

  • go test ./...

@yohamta0 yohamta0 marked this pull request as ready for review April 7, 2026 08:16
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