Skip to content

Commit

Permalink
[smithy-rs] Add RFC: Providing fallback credentials on timeout (#2218)
Browse files Browse the repository at this point in the history
* Add RFC: providing fallback credentials on timeout

* Update design/src/rfcs/rfc0031_providing_fallback_credentials_on_timeout.md

Co-authored-by: Zelda Hessler <zhessler@amazon.com>

* Update design/src/rfcs/rfc0031_providing_fallback_credentials_on_timeout.md

Co-authored-by: Zelda Hessler <zhessler@amazon.com>

* Update design/src/rfcs/rfc0031_providing_fallback_credentials_on_timeout.md

Co-authored-by: Zelda Hessler <zhessler@amazon.com>

* Update design/src/rfcs/rfc0031_providing_fallback_credentials_on_timeout.md

Co-authored-by: Zelda Hessler <zhessler@amazon.com>

* Update design/src/rfcs/rfc0031_providing_fallback_credentials_on_timeout.md

Co-authored-by: Zelda Hessler <zhessler@amazon.com>

* Update design/src/rfcs/rfc0031_providing_fallback_credentials_on_timeout.md

Co-authored-by: Zelda Hessler <zhessler@amazon.com>

* Update design/src/rfcs/rfc0031_providing_fallback_credentials_on_timeout.md

Co-authored-by: Zelda Hessler <zhessler@amazon.com>

* Update design/src/rfcs/rfc0031_providing_fallback_credentials_on_timeout.md

Co-authored-by: Zelda Hessler <zhessler@amazon.com>

* Update design/src/rfcs/rfc0031_providing_fallback_credentials_on_timeout.md

Co-authored-by: Zelda Hessler <zhessler@amazon.com>

* Update design/src/rfcs/rfc0031_providing_fallback_credentials_on_timeout.md

Co-authored-by: Zelda Hessler <zhessler@amazon.com>

* Update design/src/rfcs/rfc0031_providing_fallback_credentials_on_timeout.md

Co-authored-by: John DiSanti <jdisanti@amazon.com>

* Incorporate review feedback into RFC

This commit addresses the review feedback:
smithy-lang/smithy-rs#2218 (comment)
smithy-lang/smithy-rs#2218 (comment)
smithy-lang/smithy-rs#2218 (comment)

In addition, we have renamed the proposed method `on_timeout` to
`fallback_on_interrupt` to make it more descriptive.

* Update rfc0031_providing_fallback_credentials_on_timeout.md

* Update RFC

This commit updates RFC and leaves to discussion how `fallback_on_interrupt`
should be implemented, i.e., either as a synchronous primitive or an
asynchronous primitive.

* Update rfc0031_providing_fallback_credentials_on_timeout.md

* Update rfc0031_providing_fallback_credentials_on_timeout.md

* Update rfc0031_providing_fallback_credentials_on_timeout.md

---------

Co-authored-by: Yuki Saito <awsaito@amazon.com>
Co-authored-by: Zelda Hessler <zhessler@amazon.com>
Co-authored-by: John DiSanti <jdisanti@amazon.com>
Co-authored-by: Luca Palmieri <20745048+LukeMathWalker@users.noreply.github.com>
  • Loading branch information
5 people authored and aws-sdk-rust-ci committed Mar 30, 2023
1 parent 034b870 commit 528b5e3
Show file tree
Hide file tree
Showing 3 changed files with 31 additions and 31 deletions.
8 changes: 4 additions & 4 deletions sdk/kinesis/src/input.rs
Original file line number Diff line number Diff line change
Expand Up @@ -624,8 +624,8 @@ impl DeregisterStreamConsumerInput {
.set_endpoint(_config.endpoint_url
.clone())
.set_operation_type(Some("control".to_string()))
.set_consumer_arn(self.consumer_arn.clone())
.set_stream_arn(self.stream_arn.clone()).build()
.set_stream_arn(self.stream_arn.clone())
.set_consumer_arn(self.consumer_arn.clone()).build()
.map_err(|err|aws_smithy_http::endpoint::ResolveEndpointError::from_source("could not construct endpoint parameters", err));
let (endpoint_result, params) = match params_result {
Ok(params) => (_config.endpoint_resolver.resolve_endpoint(&params), Some(params)),
Expand Down Expand Up @@ -1014,8 +1014,8 @@ impl DescribeStreamConsumerInput {
.set_endpoint(_config.endpoint_url
.clone())
.set_operation_type(Some("control".to_string()))
.set_stream_arn(self.stream_arn.clone())
.set_consumer_arn(self.consumer_arn.clone()).build()
.set_consumer_arn(self.consumer_arn.clone())
.set_stream_arn(self.stream_arn.clone()).build()
.map_err(|err|aws_smithy_http::endpoint::ResolveEndpointError::from_source("could not construct endpoint parameters", err));
let (endpoint_result, params) = match params_result {
Ok(params) => (_config.endpoint_resolver.resolve_endpoint(&params), Some(params)),
Expand Down
48 changes: 24 additions & 24 deletions sdk/s3control/src/input.rs
Original file line number Diff line number Diff line change
Expand Up @@ -133,8 +133,8 @@ impl CreateAccessPointInput {
.clone())
.set_use_arn_region(_config.use_arn_region)
.set_requires_account_id(Some(true))
.set_account_id(self.account_id.clone())
.set_bucket(self.bucket.clone()).build()
.set_bucket(self.bucket.clone())
.set_account_id(self.account_id.clone()).build()
.map_err(|err|aws_smithy_http::endpoint::ResolveEndpointError::from_source("could not construct endpoint parameters", err));
let (endpoint_result, params) = match params_result {
Ok(params) => (_config.endpoint_resolver.resolve_endpoint(&params), Some(params)),
Expand Down Expand Up @@ -538,8 +538,8 @@ impl CreateBucketInput {
.set_endpoint(_config.endpoint_url
.clone())
.set_use_arn_region(_config.use_arn_region)
.set_outpost_id(self.outpost_id.clone())
.set_bucket(self.bucket.clone()).build()
.set_bucket(self.bucket.clone())
.set_outpost_id(self.outpost_id.clone()).build()
.map_err(|err|aws_smithy_http::endpoint::ResolveEndpointError::from_source("could not construct endpoint parameters", err));
let (endpoint_result, params) = match params_result {
Ok(params) => (_config.endpoint_resolver.resolve_endpoint(&params), Some(params)),
Expand Down Expand Up @@ -1374,8 +1374,8 @@ impl DeleteAccessPointPolicyInput {
.clone())
.set_use_arn_region(_config.use_arn_region)
.set_requires_account_id(Some(true))
.set_access_point_name(self.name.clone())
.set_account_id(self.account_id.clone()).build()
.set_account_id(self.account_id.clone())
.set_access_point_name(self.name.clone()).build()
.map_err(|err|aws_smithy_http::endpoint::ResolveEndpointError::from_source("could not construct endpoint parameters", err));
let (endpoint_result, params) = match params_result {
Ok(params) => (_config.endpoint_resolver.resolve_endpoint(&params), Some(params)),
Expand Down Expand Up @@ -2104,8 +2104,8 @@ impl DeleteBucketTaggingInput {
.clone())
.set_use_arn_region(_config.use_arn_region)
.set_requires_account_id(Some(true))
.set_account_id(self.account_id.clone())
.set_bucket(self.bucket.clone()).build()
.set_bucket(self.bucket.clone())
.set_account_id(self.account_id.clone()).build()
.map_err(|err|aws_smithy_http::endpoint::ResolveEndpointError::from_source("could not construct endpoint parameters", err));
let (endpoint_result, params) = match params_result {
Ok(params) => (_config.endpoint_resolver.resolve_endpoint(&params), Some(params)),
Expand Down Expand Up @@ -3559,8 +3559,8 @@ impl GetAccessPointPolicyInput {
.clone())
.set_use_arn_region(_config.use_arn_region)
.set_requires_account_id(Some(true))
.set_account_id(self.account_id.clone())
.set_access_point_name(self.name.clone()).build()
.set_access_point_name(self.name.clone())
.set_account_id(self.account_id.clone()).build()
.map_err(|err|aws_smithy_http::endpoint::ResolveEndpointError::from_source("could not construct endpoint parameters", err));
let (endpoint_result, params) = match params_result {
Ok(params) => (_config.endpoint_resolver.resolve_endpoint(&params), Some(params)),
Expand Down Expand Up @@ -4089,8 +4089,8 @@ impl GetBucketInput {
.clone())
.set_use_arn_region(_config.use_arn_region)
.set_requires_account_id(Some(true))
.set_account_id(self.account_id.clone())
.set_bucket(self.bucket.clone()).build()
.set_bucket(self.bucket.clone())
.set_account_id(self.account_id.clone()).build()
.map_err(|err|aws_smithy_http::endpoint::ResolveEndpointError::from_source("could not construct endpoint parameters", err));
let (endpoint_result, params) = match params_result {
Ok(params) => (_config.endpoint_resolver.resolve_endpoint(&params), Some(params)),
Expand Down Expand Up @@ -4240,8 +4240,8 @@ impl GetBucketLifecycleConfigurationInput {
.clone())
.set_use_arn_region(_config.use_arn_region)
.set_requires_account_id(Some(true))
.set_bucket(self.bucket.clone())
.set_account_id(self.account_id.clone()).build()
.set_account_id(self.account_id.clone())
.set_bucket(self.bucket.clone()).build()
.map_err(|err|aws_smithy_http::endpoint::ResolveEndpointError::from_source("could not construct endpoint parameters", err));
let (endpoint_result, params) = match params_result {
Ok(params) => (_config.endpoint_resolver.resolve_endpoint(&params), Some(params)),
Expand Down Expand Up @@ -4391,8 +4391,8 @@ impl GetBucketPolicyInput {
.clone())
.set_use_arn_region(_config.use_arn_region)
.set_requires_account_id(Some(true))
.set_bucket(self.bucket.clone())
.set_account_id(self.account_id.clone()).build()
.set_account_id(self.account_id.clone())
.set_bucket(self.bucket.clone()).build()
.map_err(|err|aws_smithy_http::endpoint::ResolveEndpointError::from_source("could not construct endpoint parameters", err));
let (endpoint_result, params) = match params_result {
Ok(params) => (_config.endpoint_resolver.resolve_endpoint(&params), Some(params)),
Expand Down Expand Up @@ -6966,8 +6966,8 @@ impl PutAccessPointPolicyInput {
.clone())
.set_use_arn_region(_config.use_arn_region)
.set_requires_account_id(Some(true))
.set_account_id(self.account_id.clone())
.set_access_point_name(self.name.clone()).build()
.set_access_point_name(self.name.clone())
.set_account_id(self.account_id.clone()).build()
.map_err(|err|aws_smithy_http::endpoint::ResolveEndpointError::from_source("could not construct endpoint parameters", err));
let (endpoint_result, params) = match params_result {
Ok(params) => (_config.endpoint_resolver.resolve_endpoint(&params), Some(params)),
Expand Down Expand Up @@ -7251,8 +7251,8 @@ impl PutBucketLifecycleConfigurationInput {
.clone())
.set_use_arn_region(_config.use_arn_region)
.set_requires_account_id(Some(true))
.set_bucket(self.bucket.clone())
.set_account_id(self.account_id.clone()).build()
.set_account_id(self.account_id.clone())
.set_bucket(self.bucket.clone()).build()
.map_err(|err|aws_smithy_http::endpoint::ResolveEndpointError::from_source("could not construct endpoint parameters", err));
let (endpoint_result, params) = match params_result {
Ok(params) => (_config.endpoint_resolver.resolve_endpoint(&params), Some(params)),
Expand Down Expand Up @@ -7447,8 +7447,8 @@ impl PutBucketPolicyInput {
.clone())
.set_use_arn_region(_config.use_arn_region)
.set_requires_account_id(Some(true))
.set_account_id(self.account_id.clone())
.set_bucket(self.bucket.clone()).build()
.set_bucket(self.bucket.clone())
.set_account_id(self.account_id.clone()).build()
.map_err(|err|aws_smithy_http::endpoint::ResolveEndpointError::from_source("could not construct endpoint parameters", err));
let (endpoint_result, params) = match params_result {
Ok(params) => (_config.endpoint_resolver.resolve_endpoint(&params), Some(params)),
Expand Down Expand Up @@ -7626,8 +7626,8 @@ impl PutBucketTaggingInput {
.clone())
.set_use_arn_region(_config.use_arn_region)
.set_requires_account_id(Some(true))
.set_bucket(self.bucket.clone())
.set_account_id(self.account_id.clone()).build()
.set_account_id(self.account_id.clone())
.set_bucket(self.bucket.clone()).build()
.map_err(|err|aws_smithy_http::endpoint::ResolveEndpointError::from_source("could not construct endpoint parameters", err));
let (endpoint_result, params) = match params_result {
Ok(params) => (_config.endpoint_resolver.resolve_endpoint(&params), Some(params)),
Expand Down
6 changes: 3 additions & 3 deletions versions.toml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
smithy_rs_revision = 'f7c417c58e0544c67b3f32df30d60d96888c9ea7'
smithy_rs_revision = 'c21705852c43fb089d33df5c46831b6d09e1475e'
aws_doc_sdk_examples_revision = '97a177aab8c3d2fef97416cb66e4b4d0da840138'

[manual_interventions]
Expand Down Expand Up @@ -1057,7 +1057,7 @@ model_hash = '9ec8a92782fbef7e2cf07fc9b5e656aa37b59668baff33acae10f270cfff4193'
[crates.aws-sdk-kinesis]
category = 'AwsSdk'
version = '0.25.0'
source_hash = '778ff0f16d712ced202ad5195b56d4609cca9579788706bd47ba8d44f45ae4f1'
source_hash = '271074bab696c6656542690ac7a030ab7b4459bbe37b3fcbb8ed2bb50a918b47'
model_hash = '5eaef8321cea414140d87258e714988d88f3e41d9cdf4b83a1417027cb702cb9'

[crates.aws-sdk-kinesisanalytics]
Expand Down Expand Up @@ -1669,7 +1669,7 @@ model_hash = 'fb47cfd49cf2d9250063914599f703ed365b51be373988dc3f2cd64321c9583b'
[crates.aws-sdk-s3control]
category = 'AwsSdk'
version = '0.25.0'
source_hash = 'e6c68ef11d443a4cdcaa1abbf28e3f5563bc849b187b599c111cbd0c552093fb'
source_hash = 'ceeac94d3982fe8b3181b88d760b7b762056612090ebaebcb539522353c10c2f'
model_hash = '930bd33c05c393293591ae01cbdb6637fb5062ed3a3ad5e2dadddd8108cc8478'

[crates.aws-sdk-s3outposts]
Expand Down

0 comments on commit 528b5e3

Please sign in to comment.