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.

Sign up for GitHub

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

Jump to bottom

Allow setting IgnoreConfiguredEndpoints on config.LoadOptions #2423

Open
1 of 2 tasks
gdavison opened this issue Dec 11, 2023 · 0 comments
Open
1 of 2 tasks

Allow setting IgnoreConfiguredEndpoints on config.LoadOptions #2423

gdavison opened this issue Dec 11, 2023 · 0 comments
Labels
feature-request A feature should be added or improved. p3 This is a minor priority issue queued This issues is on the AWS team's backlog

Comments

@gdavison
Copy link
Contributor

Describe the feature

The config.SharedConfig and config.EnvConfig structs both have a parameter IgnoreConfiguredEndpoints that sets a common endpoint URL for all services. config.LoadOptions does not have the corresponding parameter.

Also add the corresponding WithIgnoreConfiguredEndpoints functional option.

Use Case

There does not seem to be a way to override endpoints configured using environment variables or the shared config file. While the aws.Config returned by config.LoadDefaultConfig could be manually modified, this would not affect any service clients potentially used internally by LoadDefaultConfig, such as for STS.

Proposed Solution

No response

Other Information

No response

Acknowledgements

  • I may be able to implement this feature request
  • This feature might incur a breaking change

AWS Go SDK V2 Module Versions Used

github.com/aws/aws-sdk-go-v2 v1.24.0
github.com/aws/aws-sdk-go-v2/config v1.26.1

Go version used

go version go1.21.0 darwin/arm64
@gdavison gdavison added feature-request A feature should be added or improved. needs-triage This issue or PR still needs to be triaged. labels Dec 11, 2023
@gdavison gdavison mentioned this issue Dec 11, 2023
Open
@lucix-aws lucix-aws added p3 This is a minor priority issue queued This issues is on the AWS team's backlog and removed needs-triage This issue or PR still needs to be triaged. labels Dec 11, 2023
@gdavison gdavison mentioned this issue Dec 11, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature-request A feature should be added or improved. p3 This is a minor priority issue queued This issues is on the AWS team's backlog
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Adds IgnoreConfiguredEndpoints and BaseEndpoint to config.LoadOptions gdavison/aws-sdk-go-v2
2 participants
@gdavison @lucix-aws