respond to PR comments

smithy-lang · Dec 2, 2024 · ea72d21 · ea72d21
1 parent 02f1da6
commit ea72d21
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/aws/rust-runtime/Cargo.lock b/aws/rust-runtime/Cargo.lock
diff --git a/aws/sdk/integration-tests/s3/tests/signing-it.rs b/aws/sdk/integration-tests/s3/tests/signing-it.rs
@@ -72,8 +72,10 @@ async fn disable_payload_signing_works() {
     assert_eq!("UNSIGNED-PAYLOAD", x_amz_content_sha256);
 }
 
-// This test ensures that the interceptor's payload signing setting
-// takes priority over the runtime plugin for disabling signing.
+// This test ensures that the request checksum interceptor payload signing
+// override takes priority over the runtime plugin's override. If it didn't,
+// then disabling payload signing would cause requests to incorrectly omit
+// trailers.
 #[tokio::test]
 async fn disable_payload_signing_works_with_checksums() {
     let (http_client, request) = capture_request(None);