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

service/s3: Add support for regional S3 us-east-1 endpoint #2938

Merged
merged 2 commits into from
Nov 14, 2019

Conversation

jasdel
Copy link
Contributor

@jasdel jasdel commented Nov 9, 2019

Adds support for S3 configuring an SDK Amazon S3 client for the regional us-east-1 endpoint instead of the default global s3 endpoint.

Adds a new configuration option, S3UsEast1RegionalEndpoint which when set to RegionalS3UsEast1Endpoint, and region is us-east-1 the S3 client will resolve the us-east-1 regional endpoint, s3.us-east-1.amazonaws.com instead of the global S3 endpoint, s3.amazonaws.com. The SDK defaults to the current global S3 endpoint resolution for backwards compatible.

Opt-in to the us-east-1 regional endpoint via the SDK's Config, environment variable, AWS_S3_US_EAST_1_REGIONAL_ENDPOINT=regional, or shared config option, s3_us_east_1_regional_endpoint=regional.

Note the SDK does not support the shared configuration file by default. You must opt-in to that behavior via Session Option SharedConfigState, or AWS_SDK_LOAD_CONFIG=true environment variable.

@jasdel jasdel added the needs-review This issue or pull request needs review from a core team member. label Nov 9, 2019
@jasdel jasdel self-assigned this Nov 9, 2019
@jasdel jasdel force-pushed the feat/S3RegionalEndpoint branch from a0eed1e to 8be689c Compare November 9, 2019 00:42
@skmcgrail skmcgrail self-requested a review November 12, 2019 19:29
CHANGELOG_PENDING.md Outdated Show resolved Hide resolved
@jasdel jasdel force-pushed the feat/S3RegionalEndpoint branch 2 times, most recently from 55ad05e to 6ab3363 Compare November 12, 2019 23:02
@jasdel jasdel force-pushed the feat/S3RegionalEndpoint branch from 6ab3363 to 99eacad Compare November 13, 2019 00:58
Adds support for S3 configuring an SDK Amazon S3 client for the regional
us-east-1 endpoint instead of the default global s3 endpoint.

Adds a new configuration option, `S3UsEast1RegionalEndpoint` which when
set to RegionalS3UsEast1Endpoint, and region is `us-east-1` the S3
client will resolve the `us-east-1` regional endpoint,
`s3.us-east-1.amazonaws.com` instead of the global S3 endpoint,
`s3.amazonaws.com`. The SDK defaults to the current global S3 endpoint
resolution for backwards compatibility.

Opt-in to the `us-east-1` regional endpoint via the SDK's Config,
environment variable, `AWS_S3_US_EAST_1_REGIONAL_ENDPOINT=regional`, or
shared config option, `s3_us_east_1_regional_endpoint=regional`.

Note the SDK does not support the shared configuration file by default.
You must opt-in to that behavior via Session Option `SharedConfigState`,
or `AWS_SDK_LOAD_CONFIG=true` environment variable.
@jasdel jasdel force-pushed the feat/S3RegionalEndpoint branch from 99eacad to 26036b8 Compare November 14, 2019 00:29
@jasdel jasdel removed the needs-review This issue or pull request needs review from a core team member. label Nov 14, 2019
@jasdel jasdel merged commit a12a190 into aws:master Nov 14, 2019
@jasdel jasdel deleted the feat/S3RegionalEndpoint branch November 14, 2019 00:57
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