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

[S3][AccessPoints] Custom endpoints should not be modified when using accesspoints. #479

Closed
skmcgrail opened this issue Jan 28, 2020 · 2 comments
Labels

Comments

@skmcgrail
Copy link
Member

The current implementation in the V2 SDK for support S3's AccessPoint feature will manipulate the resolved endpoint URL when an AccessPoint ARN is provided in a bucket parameter. This is not the intended behavior per the implementation in V1. This is due to the current endpoint resolver design preventing the ability to disambiguate an endpoint resolved from the default endpoint resolver in aws/endpoint from a custom endpoint provided by a user using a custom resolver such as aws/endpoints.EndpointResolverFunc

jasdel added a commit that referenced this issue Oct 27, 2020
…oint or not

Adds a new member to the `aws.Endpoint` struct, `HostnameImmutable`. This member
directs if the SDK is allowed to modify the resolved endpoint to meet the
requirements of the API.  If `HostnameImmutable` is true, the SDK will not
attempt to modify the hostname via any customizations, or prefix behavior. Flag
 defaults to false.

- Fixes #827 (SDK requiring region for endpoint resolution)
- Fixes #328
- Fixes #364
- Related to #836 When Amazon S3 Accesspoint support is implemented.
- Related to #479

Depends on aws/smithy-go#222
jasdel added a commit that referenced this issue Oct 27, 2020
…oint or not

Adds a new member to the `aws.Endpoint` struct, `HostnameImmutable`. This member
directs if the SDK is allowed to modify the resolved endpoint to meet the
requirements of the API.  If `HostnameImmutable` is true, the SDK will not
attempt to modify the hostname via any customizations, or prefix behavior. Flag
 defaults to false.

- Fixes #827 (SDK requiring region for endpoint resolution)
- Fixes #328
- Fixes #364
- Related to #836 When Amazon S3 Accesspoint support is implemented.
- Related to #479

Depends on aws/smithy-go#222
jasdel added a commit that referenced this issue Oct 27, 2020
Adds a new member to the `aws.Endpoint` struct, `HostnameImmutable`. This member directs if the SDK is allowed to modify the resolved endpoint to meet the requirements of the API.  If `HostnameImmutable` is true, the SDK will not attempt to modify the hostname via any customizations, or prefix behavior. Flag defaults to false.

- Fixes #827 (SDK requiring region for endpoint resolution)
- Fixes #328
- Fixes #364
- Related to #836 When Amazon S3 Accesspoint support is implemented.
- Related to #479

Depends on aws/smithy-go#222
@jasdel
Copy link
Contributor

jasdel commented Nov 11, 2020

This issue was resolved with #848 that added an option to the Endpoint resolver for users to direct the SDK to not modify the custom endpoint.

@jasdel jasdel closed this as completed Nov 11, 2020
@github-actions
Copy link

⚠️COMMENT VISIBILITY WARNING⚠️

Comments on closed issues are hard for our team to see.
If you need more assistance, please either tag a team member or open a new issue that references this one.
If you wish to keep having a conversation with other community members under this issue feel free to do so.

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

No branches or pull requests

2 participants