Skip to content

fix(storage): Fix encoding of space characters in file names in signed_url v4#7860

Merged
quartzmo merged 2 commits into
googleapis:masterfrom
quartzmo:storage-signed-url-v4-fix
Oct 2, 2020
Merged

fix(storage): Fix encoding of space characters in file names in signed_url v4#7860
quartzmo merged 2 commits into
googleapis:masterfrom
quartzmo:storage-signed-url-v4-fix

Conversation

@quartzmo
Copy link
Copy Markdown
Member

@quartzmo quartzmo commented Oct 2, 2020

The acceptance test coverage for this change passes for me locally. Updates to the conformance tests to provide unit test coverage for this change are still pending.

closes: #7835

@quartzmo quartzmo requested a review from a team October 2, 2020 19:45
@quartzmo quartzmo self-assigned this Oct 2, 2020
@google-cla google-cla Bot added the cla: yes This human has signed the Contributor License Agreement. label Oct 2, 2020
@quartzmo quartzmo added the api: storage Issues related to the Cloud Storage API. label Oct 2, 2020
@quartzmo quartzmo requested a review from frankyn October 2, 2020 19:46
@quartzmo
Copy link
Copy Markdown
Member Author

quartzmo commented Oct 2, 2020

/cc @chvreddy

@quartzmo quartzmo merged commit 4380c02 into googleapis:master Oct 2, 2020
@chvreddy
Copy link
Copy Markdown

chvreddy commented Oct 2, 2020

Thanks @quartzmo @frankyn
So when can we expect the release with this bug?

@quartzmo
Copy link
Copy Markdown
Member Author

quartzmo commented Oct 5, 2020

@chvreddy This fix was released today in google-cloud-storage v1.29.1, thank you!

@chvreddy
Copy link
Copy Markdown

chvreddy commented Oct 5, 2020

@quartzmo Thank you very much!

@quartzmo
Copy link
Copy Markdown
Member Author

quartzmo commented Oct 5, 2020

@chvreddy Very happy that you brought this to our attention! Please reopen your original issue if the fix doesn't work for you or isn't complete. And always feel free to open issues for anything that needs to be improved. Thanks!

@quartzmo quartzmo deleted the storage-signed-url-v4-fix branch October 5, 2020 19:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: storage Issues related to the Cloud Storage API. cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Signed URL: Special Chars in file name not handled properly in V4 version

3 participants