fix: resolve endpoint in command serializer

The endpoint used to be a function returning a promise of Endpoint,
and it is resolved in serializer parameter. It introduces mismatch
between SerDeContext type and the value, where the value contains
resolved endpoint but the type is a provider function returning Endpoint
promise. This change assumes the SerDeContext.endpoint is actually
a provider function and only resolve the promise when constructing
a request.

This change also fix the issue that the request resolved path got
overwritten by '/'. Now only the hostname, port and scheme from
client endpoint config is persisted when creating a request.

Author: AllanZhengYP

smithy-typescript-codegen/src/main/java/software/amazon/smithy/typescript/codegen/integration/HttpBindingProtocolGenerator.java

@@ -227,11 +227,14 @@ private void generateOperationSerializer(
                 HttpProtocolGeneratorUtils.writeHostPrefix(context, operation);
             }
 
+            writer.write("const {hostname, protocol, port} = await context.endpoint();");
             writer.openBlock("return new $T({", "});", requestType, () -> {
                 if (hasHostPrefix) {
                     writer.write("hostname: resolvedHostname,");
                 }
-                writer.write("protocol: \"https\",");
+                writer.write("protocol: \"https\" || protocol,");
+                writer.write("hostname,");
+                writer.write("port,");
                 writer.write("method: $S,", trait.getMethod());
                 writer.write("headers: headers,");
                 writer.write("path: resolvedPath,");
@@ -247,7 +250,6 @@ private void generateOperationSerializer(
                 }
                 // Always set the body,
                 writer.write("body: body,");
-                writer.write("...context.endpoint,");
             });
         });
 

smithy-typescript-codegen/src/main/java/software/amazon/smithy/typescript/codegen/integration/HttpRpcProtocolGenerator.java

@@ -100,19 +100,21 @@ public void generateSharedComponents(GenerationContext context) {
         writer.addUseImports(requestType);
         writer.addImport("SerdeContext", "__SerdeContext", "@aws-sdk/types");
         writer.addImport("HeaderBag", "__HeaderBag", "@aws-sdk/types");
-        writer.openBlock("const buildHttpRpcRequest = (\n"
+        writer.openBlock("const buildHttpRpcRequest = async (\n"
                        + "  context: __SerdeContext,\n"
                        + "  headers: __HeaderBag,\n"
                        + "  path: string,\n"
                        + "  resolvedHostname: string | undefined,\n"
                        + "  body: any,\n"
-                       + "): $T => {", "};", requestType, () -> {
+                       + "): Promise<$T> => {", "};", requestType, () -> {
+            writer.write("const {hostname, protocol, port} = await context.endpoint();");
             writer.openBlock("const contents: any = {", "};", () -> {
-                writer.write("protocol: \"https\",");
+                writer.write("protocol: \"https\" || protocol,");
+                writer.write("hostname,");
+                writer.write("port,");
                 writer.write("method: \"POST\",");
                 writer.write("path: path,");
                 writer.write("headers: headers,");
-                writer.write("...context.endpoint,");
             });
             writer.openBlock("if (resolvedHostname !== undefined) {", "}", () -> {
                 writer.write("contents.hostname = resolvedHostname;");