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 for region discovery issue with aws sdkv2 when running in ec2 #2977

Merged
merged 1 commit into from
Sep 9, 2024

Conversation

wolfeidau
Copy link
Contributor

Description

The new KMS signing support uses aws sdkv2, this automatically discovers region from environment or local configuration files, however it doesn't automatically use imds. This code replicates the v1 local configuration preference with fall back to imds service.

Context

Changes

Testing

  • Tests have run locally (with go test ./...). Buildkite employees may check this if the pipeline has run automatically.
  • Code is formatted (with go fmt ./...)

@wolfeidau wolfeidau force-pushed the fix_aws_sdkv2_region_discovery branch from 4bf82d5 to 8cb13c0 Compare September 9, 2024 04:54
Copy link
Contributor

@DrJosh9000 DrJosh9000 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@wolfeidau wolfeidau force-pushed the fix_aws_sdkv2_region_discovery branch from 8cb13c0 to 90b7d65 Compare September 9, 2024 05:14
@wolfeidau wolfeidau merged commit 5394c21 into main Sep 9, 2024
1 check passed
@wolfeidau wolfeidau deleted the fix_aws_sdkv2_region_discovery branch September 9, 2024 05:46
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