fix(codegen): use wrapped response when deserializing modeled exceptions (#358)

aajtodd · web-flow · commit 9f9353cb3950 · 2021-10-21T08:34:19.000-04:00
diff --git a/codegen/smithy-aws-kotlin-codegen/src/main/kotlin/aws/sdk/kotlin/codegen/customization/s3/S3Generator.kt b/codegen/smithy-aws-kotlin-codegen/src/main/kotlin/aws/sdk/kotlin/codegen/customization/s3/S3Generator.kt
@@ -114,7 +114,7 @@ class S3Generator : RestXml() {
             }
 
             writer.write("")
-                .write("val modeledException = modeledExceptionDeserializer.deserialize(context, response)")
+                .write("val modeledException = modeledExceptionDeserializer.deserialize(context, wrappedResponse)")
                 .write("#T(modeledException, wrappedResponse, errorDetails)", setS3ErrorMetadata)
                 .write("throw modeledException")
         }
diff --git a/codegen/smithy-aws-kotlin-codegen/src/main/kotlin/aws/sdk/kotlin/codegen/protocols/core/AwsHttpBindingProtocolGenerator.kt b/codegen/smithy-aws-kotlin-codegen/src/main/kotlin/aws/sdk/kotlin/codegen/protocols/core/AwsHttpBindingProtocolGenerator.kt
@@ -155,7 +155,7 @@ abstract class AwsHttpBindingProtocolGenerator : HttpBindingProtocolGenerator()
             }
 
             writer.write("")
-                .write("val modeledException = modeledExceptionDeserializer.deserialize(context, response)")
+                .write("val modeledException = modeledExceptionDeserializer.deserialize(context, wrappedResponse)")
                 .write("#T(modeledException, wrappedResponse, errorDetails)", AwsRuntimeTypes.Http.setAseErrorMetadata)
                 .write("throw modeledException")
         }

Original file line number	Diff line number	Diff line change
`@@ -114,7 +114,7 @@ class S3Generator : RestXml() {`
`114`	`114`	`}`
`115`	`115`
`116`	`116`	`writer.write("")`
`117`		`- .write("val modeledException = modeledExceptionDeserializer.deserialize(context, response)")`
	`117`	`+ .write("val modeledException = modeledExceptionDeserializer.deserialize(context, wrappedResponse)")`
`118`	`118`	`.write("#T(modeledException, wrappedResponse, errorDetails)", setS3ErrorMetadata)`
`119`	`119`	`.write("throw modeledException")`
`120`	`120`	`}`
Original file line number	Diff line number	Diff line change
`@@ -155,7 +155,7 @@ abstract class AwsHttpBindingProtocolGenerator : HttpBindingProtocolGenerator()`
`155`	`155`	`}`
`156`	`156`
`157`	`157`	`writer.write("")`
`158`		`- .write("val modeledException = modeledExceptionDeserializer.deserialize(context, response)")`
	`158`	`+ .write("val modeledException = modeledExceptionDeserializer.deserialize(context, wrappedResponse)")`
`159`	`159`	`.write("#T(modeledException, wrappedResponse, errorDetails)", AwsRuntimeTypes.Http.setAseErrorMetadata)`
`160`	`160`	`.write("throw modeledException")`
`161`	`161`	`}`