feat: extended URI encoding#915
Conversation
kstich
left a comment
There was a problem hiding this comment.
There are other locations that will need their uri encoding function updated in smithy-typescript as well after this.
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
Codecov Report
@@ Coverage Diff @@
## smithy-codegen #915 +/- ##
===============================================
Coverage 92.12% 92.12%
===============================================
Files 145 145
Lines 2908 2908
Branches 513 513
===============================================
Hits 2679 2679
Misses 229 229Continue to review full report at Codecov.
|
This comment has been minimized.
This comment has been minimized.
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
|
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs and link to relevant comments in this thread. |
Fixes: #866
Adds extendedEncodeURI function to wrap encodeURIComponent and encode additional characters, !, ', (, ), and *, to adhere to RFC 3986.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.