fix: correctly include custom ports in signing/presigning

smithy-lang / smithy-kotlin

Smithy code generator for Kotlin (in development)

Apache License 2.0

71 stars 26 forks source link

fix: correctly include custom ports in signing/presigning #1026

Closed ianbotsf closed 5 months ago

ianbotsf commented 5 months ago

Issue \

https://github.com/awslabs/aws-sdk-kotlin/issues/1177

Description of changes

This change adds handling of non-standard ports during signing and presigning by including them in the Host header and also the canonical string used in signature calculation.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

sonarcloud[bot] commented 5 months ago

Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud