amzn / smoke-http

Specialised HTTP Client for service operations abstracted from the HTTP protocol.
Apache License 2.0
71 stars 15 forks source link

Encode the endpoint path prior to decorating the log context #120

Closed pbthif closed 1 year ago

pbthif commented 1 year ago

Issue #, if available:

Description of changes:

During outgoing requests, if the endpoint path contains tokens, the outgoingEndpoint log decoration would contain the unresolved tokens (ex. "https://s3.us-west-2.amazonaws.com:443/%7BBucket%7D/%7BKey+%7D"). This change ensure that the log is decorated with the encoded version of the endpoint path.

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