Skip to content

Commit f392e8c

Browse files
rcohjdisanti
authored andcommitted
[smithy-rs] Add credentials exposure test & fix STS + SSO (#2603)
## Motivation and Context - credentials providers may leak credentials in the HTTP body at the debug level ## Description This adds a test to aws-config that looks for leaked credentials in all of our provider integration tests—since these test use AWS APIs under the hood, this also serves to test AWS services in general. To support this, `sensitive` was added to the ParseHttpResponse trait and code was generated to take action based on this change. - [x] Add environment variable to force logging of the body - [x] consider if we want to suppress request body logging as well ## Testing <!--- Please describe in detail how you tested your changes --> <!--- Include details of your testing environment, and the tests you ran to --> <!--- see how your change affects other areas of the code, etc. --> ## Checklist <!--- If a checkbox below is not applicable, then please DELETE it rather than leaving it unchecked --> - [x] I have updated `CHANGELOG.next.toml` if I made changes to the smithy-rs codegen or runtime crates - [x] I have updated `CHANGELOG.next.toml` if I made changes to the AWS SDK, generated SDK code, or SDK runtime crates ---- _By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice._ --------- Co-authored-by: John DiSanti <jdisanti@amazon.com>
1 parent 86a6647 commit f392e8c

File tree

13,490 files changed

+27134
-9645
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

13,490 files changed

+27134
-9645
lines changed

sdk/accessanalyzer/src/operation/apply_archive_rule.rs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -99,6 +99,7 @@ impl aws_smithy_http::response::ParseStrictResponse for ApplyArchiveRule {
9999
crate::protocol_serde::shape_apply_archive_rule::de_apply_archive_rule_http_response(status, headers, body)
100100
}
101101
}
102+
102103
}
103104

104105
/// Do not use this.

sdk/accessanalyzer/src/operation/cancel_policy_generation.rs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -98,6 +98,7 @@ impl aws_smithy_http::response::ParseStrictResponse for CancelPolicyGeneration {
9898
crate::protocol_serde::shape_cancel_policy_generation::de_cancel_policy_generation_http_response(status, headers, body)
9999
}
100100
}
101+
101102
}
102103

103104
/// Do not use this.

sdk/accessanalyzer/src/operation/create_access_preview.rs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -99,6 +99,7 @@ impl aws_smithy_http::response::ParseStrictResponse for CreateAccessPreview {
9999
crate::protocol_serde::shape_create_access_preview::de_create_access_preview_http_response(status, headers, body)
100100
}
101101
}
102+
102103
}
103104

104105
/// Do not use this.

sdk/accessanalyzer/src/operation/create_analyzer.rs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -99,6 +99,7 @@ impl aws_smithy_http::response::ParseStrictResponse for CreateAnalyzer {
9999
crate::protocol_serde::shape_create_analyzer::de_create_analyzer_http_response(status, headers, body)
100100
}
101101
}
102+
102103
}
103104

104105
/// Do not use this.

sdk/accessanalyzer/src/operation/create_archive_rule.rs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -105,6 +105,7 @@ impl aws_smithy_http::response::ParseStrictResponse for CreateArchiveRule {
105105
crate::protocol_serde::shape_create_archive_rule::de_create_archive_rule_http_response(status, headers, body)
106106
}
107107
}
108+
108109
}
109110

110111
/// Do not use this.

sdk/accessanalyzer/src/operation/delete_analyzer.rs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -111,6 +111,7 @@ impl aws_smithy_http::response::ParseStrictResponse for DeleteAnalyzer {
111111
crate::protocol_serde::shape_delete_analyzer::de_delete_analyzer_http_response(status, headers, body)
112112
}
113113
}
114+
114115
}
115116

116117
/// Do not use this.

sdk/accessanalyzer/src/operation/delete_archive_rule.rs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -117,6 +117,7 @@ impl aws_smithy_http::response::ParseStrictResponse for DeleteArchiveRule {
117117
crate::protocol_serde::shape_delete_archive_rule::de_delete_archive_rule_http_response(status, headers, body)
118118
}
119119
}
120+
120121
}
121122

122123
/// Do not use this.

sdk/accessanalyzer/src/operation/get_access_preview.rs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -109,6 +109,7 @@ impl aws_smithy_http::response::ParseStrictResponse for GetAccessPreview {
109109
crate::protocol_serde::shape_get_access_preview::de_get_access_preview_http_response(status, headers, body)
110110
}
111111
}
112+
112113
}
113114

114115
/// Do not use this.

sdk/accessanalyzer/src/operation/get_analyzed_resource.rs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -109,6 +109,7 @@ impl aws_smithy_http::response::ParseStrictResponse for GetAnalyzedResource {
109109
crate::protocol_serde::shape_get_analyzed_resource::de_get_analyzed_resource_http_response(status, headers, body)
110110
}
111111
}
112+
112113
}
113114

114115
/// Do not use this.

sdk/accessanalyzer/src/operation/get_analyzer.rs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -98,6 +98,7 @@ impl aws_smithy_http::response::ParseStrictResponse for GetAnalyzer {
9898
crate::protocol_serde::shape_get_analyzer::de_get_analyzer_http_response(status, headers, body)
9999
}
100100
}
101+
101102
}
102103

103104
/// Do not use this.

0 commit comments

Comments
 (0)