8000 codegen: Fix EventSteam code generation bug (#1638) · aws/aws-sdk-go-v2@cd1c8a3 · GitHub
[go: up one dir, main page]
Skip to content

Commit cd1c8a3

Browse files
skmcgrailskmcgrail
authored
codegen: Fix EventSteam code generation bug (#1638)
* codegen: Fix EventSteam deserializer generation bug * codegen: Namespace event stream writer and reader implementations * Regnerated Clients
1 parent 5399c50 commit cd1c8a3

File tree

8 files changed

+500
-496
lines changed

8 files changed

+500
-496
lines changed

‎codegen/smithy-aws-go-codegen/src/main/java/software/amazon/smithy/aws/go/codegen/AwsEventStreamUtils.java‎

Lines changed: 403 additions & 400 deletions
Large diffs are not rendered by default.

‎codegen/smithy-aws-go-codegen/src/main/java/software/amazon/smithy/aws/go/codegen/JsonRpcProtocolGenerator.java‎

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -203,7 +203,8 @@ public void generateEventStreamComponents(GenerationContext context) {
203203

204204
@Override
205205
protected void writeOperationSerializerMiddlewareEventStreamSetup(GenerationContext context, EventStreamInfo info) {
206-
AwsEventStreamUtils.writeOperationSerializerMiddlewareEventStreamSetup(context, info);
206+
AwsEventStreamUtils.writeOperationSerializerMiddlewareEventStreamSetup(context, info,
207+
"httpBindingEncoder");
207208
}
208209

209210
@Override
@@ -315,7 +316,7 @@ protected void generateEventStreamDeserializers(
315316
payloadTarget, ctx.getService(), getProtocolName());
316317
var ctxWriter = ctx.getWriter().get();
317318
ctxWriter.openBlock("if err := $L(&$L, shape); err != nil {", "}", functionName, operand,
318-
() -> handleDecodeError(ctxWriter))
319+
() -> handleDecodeError(ctxWriter))
319320
.write("return nil");
320321
});
321322

@@ -348,7 +349,7 @@ protected void generateEventStreamDeserializers(
348349
AwsProtocolUtils.initializeJsonEventMessageDeserializer(ctx, "nil,");
349350
var ctxWriter = ctx.getWriter().get();
350351
ctxWriter.openBlock("if err := $L(&$L, shape); err != nil {", "}", functionName, operand,
351-
() -> handleDecodeError(ctxWriter, "nil,"))
352+
() -> handleDecodeError(ctxWriter, "nil,"))
352353
.write("return v, nil");
353354
});
354355
var initialMessageMembers = streamInfo.getInitialMessageMembers()

‎codegen/smithy-aws-go-codegen/src/main/java/software/amazon/smithy/aws/go/codegen/RestJsonProtocolGenerator.java‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -519,7 +519,7 @@ public void generateEventStreamComponents(GenerationContext context) {
519519

520520
@Override
521521
protected void writeOperationSerializerMiddlewareEventStreamSetup(GenerationContext context, EventStreamInfo info) {
522-
AwsEventStreamUtils.writeOperationSerializerMiddlewareEventStreamSetup(context, info);
522+
AwsEventStreamUtils.writeOperationSerializerMiddlewareEventStreamSetup(context, info, "restEncoder");
523523
}
524524

525525
@Override

‎codegen/smithy-aws-go-codegen/src/main/java/software/amazon/smithy/aws/go/codegen/RestXmlProtocolGenerator.java‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -475,7 +475,7 @@ public void generateEventStreamComponents(GenerationContext context) {
475475

476476
@Override
477477
protected void writeOperationSerializerMiddlewareEventStreamSetup(GenerationContext context, EventStreamInfo info) {
478-
AwsEventStreamUtils.writeOperationSerializerMiddlewareEventStreamSetup(context, info);
478+
AwsEventStreamUtils.writeOperationSerializerMiddlewareEventStreamSetup(context, info, "restEncoder");
479479
}
480480

481481
@Override

‎service/kinesis/eventstream.go‎

Lines changed: 12 additions & 12 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎service/lexruntimev2/eventstream.go‎

Lines changed: 27 additions & 27 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)
0