Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Merged
merged 2 commits into from
Oct 2, 2020

Conversation

quartzmo
Copy link
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 as a code owner 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
Member Author

quartzmo commented Oct 2, 2020

/cc @chvreddy

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

chvreddy commented Oct 2, 2020

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

@quartzmo
Copy link
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

chvreddy commented Oct 5, 2020

@quartzmo Thank you very much!

@quartzmo
Copy link
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