Closed trivikr closed 1 month ago
The protocol tests do not verify resolvedHost from httpRequestTests
resolvedHost
Update codegen to validate resolvedHost, likely in https://github.com/smithy-lang/smithy-typescript/blob/main/smithy-typescript-codegen/src/main/java/software/amazon/smithy/typescript/codegen/HttpProtocolTestGenerator.java
N/A
Noticed while adding machinelearning test in https://github.com/aws/aws-sdk-js-v3/pull/6161
Other tests which use resolvedHost: https://github.com/search?q=repo%3Asmithy-lang%2Fsmithy%20resolvedHost%20path%3A*.smithy&type=code
Is your feature request related to a problem? Please describe.
The protocol tests do not verify
resolvedHost
from httpRequestTestsDescribe the solution you'd like
Update codegen to validate
resolvedHost
, likely in https://github.com/smithy-lang/smithy-typescript/blob/main/smithy-typescript-codegen/src/main/java/software/amazon/smithy/typescript/codegen/HttpProtocolTestGenerator.javaDescribe alternatives you've considered
N/A
Additional context
Noticed while adding machinelearning test in https://github.com/aws/aws-sdk-js-v3/pull/6161
resolvedHost
is defined in https://github.com/smithy-lang/smithy/blob/main/smithy-aws-protocol-tests/model/awsJson1_1/services/machinelearning.smithy#L44