fix: override signing config for s3 control (#1188)

awslabs · Jan 23, 2024 · 892336a · 892336a
1 parent cef72c5
commit 892336a
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 2 deletions.
diff --git a/.changes/8147aee6-9f81-40cc-965e-9867044439b9.json b/.changes/8147aee6-9f81-40cc-965e-9867044439b9.json
@@ -0,0 +1,9 @@
+{
+    "id": "8147aee6-9f81-40cc-965e-9867044439b9",
+    "type": "bugfix",
+    "description": "Add missing `x-amz-content-sha256` header for SigV4 requests.",
+    "issues": [
+        "awslabs/aws-sdk-kotlin#1187"
+    ],
+    "module": "s3control"
+}
diff --git a/...ws-sdk-codegen/src/main/kotlin/aws/sdk/kotlin/codegen/customization/s3/S3SigningConfig.kt b/...ws-sdk-codegen/src/main/kotlin/aws/sdk/kotlin/codegen/customization/s3/S3SigningConfig.kt
@@ -24,8 +24,10 @@ class S3SigningConfig : KotlinIntegration {
     override val order: Byte
         get() = 127
 
-    override fun enabledForService(model: Model, settings: KotlinSettings) =
-        model.expectShape<ServiceShape>(settings.service).isS3
+    override fun enabledForService(model: Model, settings: KotlinSettings): Boolean {
+        val service = model.expectShape<ServiceShape>(settings.service)
+        return (service.isS3 || service.isS3Control)
+    }
 
     override val sectionWriters: List<SectionWriterBinding>
         get() = listOf(