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

jewel: rgw: replace '+' with "%20" in canonical query string for s3 v4 auth. #16951

Merged
merged 1 commit into from Sep 7, 2017

Conversation

Projects
None yet
5 participants
@mattbenjamin
Contributor

mattbenjamin commented Aug 9, 2017

@mattbenjamin mattbenjamin added this to the jewel milestone Aug 9, 2017

@mattbenjamin mattbenjamin self-assigned this Aug 9, 2017

@mattbenjamin mattbenjamin requested review from oritwas and mdw-at-linuxbox Aug 9, 2017

@oritwas

oritwas approved these changes Aug 9, 2017

@branch-predictor

This comment has been minimized.

Show comment
Hide comment
@branch-predictor

branch-predictor Aug 10, 2017

Member

That civetweb submodule change is intentional?

Member

branch-predictor commented Aug 10, 2017

That civetweb submodule change is intentional?

@oritwas

This comment has been minimized.

Show comment
Hide comment
@oritwas
Contributor

oritwas commented Aug 10, 2017

rgw: replace '+' with "%20" in canonical query string for s3 v4 auth.
Fixes: http://tracker.ceph.com/issues/20501

Signed-off-by: Zhang Shaowen <zhangshaowen@cmss.chinamobile.com>
(cherry picked from commit 416bc51)
Signed-off-by: Matt Benjamin <mbenjamin@redhat.com>
@smithfarm

This comment has been minimized.

Show comment
Hide comment
@smithfarm

smithfarm Sep 7, 2017

Contributor

This passed an rgw suite at http://tracker.ceph.com/issues/20613#note-53

Contributor

smithfarm commented Sep 7, 2017

This passed an rgw suite at http://tracker.ceph.com/issues/20613#note-53

@smithfarm smithfarm merged commit 02fef5c into ceph:jewel Sep 7, 2017

3 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodified Submodules submodules for project are unmodified
Details
make check make check succeeded
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment