JS library has a known issue that it does not URI encode the copy source path. We have to fix it only in next major version because users may already encoded the copy source, adding URI encode for them will cause the value to be double-encoded.
Related GitHub issue #, if available:
Instructions
If this PR should not be merged upon approval for any reason, please submit as a DRAFT
Which product(s) are affected by this PR (if applicable)?
[ ] amplify-cli
[ ] amplify-ui
[ ] amplify-studio
[ ] amplify-hosting
[x] amplify-libraries
Which platform(s) are affected by this PR (if applicable)?
[x] JS
[ ] Swift
[ ] Android
[ ] Flutter
[x] React Native
Please add the product(s)/platform(s) affected to the PR title
[ ] Does this PR include filetypes other than markdown or images? Please add or update unit tests accordingly.
[ ] Are any files being deleted with this PR? If so, have the needed redirects been created?
[ ] Are all links in MDX files using the MDX link syntax rather than HTML link syntax? ref: MDX: [link](https://docs.amplify.aws/)
HTML: <a href="https://docs.amplify.aws/">link</a>
When this PR is ready to merge, please check the box below
[x] Ready to merge
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Description of changes:
JS library has a known issue that it does not URI encode the copy source path. We have to fix it only in next major version because users may already encoded the copy source, adding URI encode for them will cause the value to be double-encoded.
Related GitHub issue #, if available:
Instructions
If this PR should not be merged upon approval for any reason, please submit as a DRAFT
Which product(s) are affected by this PR (if applicable)?
Which platform(s) are affected by this PR (if applicable)?
Please add the product(s)/platform(s) affected to the PR title
Checks
[x] Does this PR conform to the styleguide?
[ ] Does this PR include filetypes other than markdown or images? Please add or update unit tests accordingly.
[ ] Are any files being deleted with this PR? If so, have the needed redirects been created?
[ ] Are all links in MDX files using the MDX link syntax rather than HTML link syntax?
ref: MDX:
[link](https://docs.amplify.aws/)
HTML:<a href="https://docs.amplify.aws/">link</a>
When this PR is ready to merge, please check the box below
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.