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

rgw: returns requested bucket name raw in Bucket response header #5715

Merged
merged 1 commit into from Sep 24, 2015
Merged

rgw: returns requested bucket name raw in Bucket response header #5715

merged 1 commit into from Sep 24, 2015

Conversation

ghost
Copy link

@ghost ghost commented Aug 30, 2015

Fixes: #12537
Don't send the bucket name back without url encoding it.

Signed-off-by: Yehuda Sadeh <yehuda@redhat.com>
(cherry picked from commit ad5507f)
@ghost ghost self-assigned this Aug 30, 2015
@ghost ghost added this to the hammer milestone Aug 30, 2015
@ghost ghost added bug-fix rgw labels Aug 30, 2015
ghost pushed a commit that referenced this pull request Aug 30, 2015
…cket response header

Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Aug 30, 2015
…cket response header

Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Aug 30, 2015
…cket response header

Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Aug 31, 2015
…cket response header

Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Aug 31, 2015
…cket response header

Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Aug 31, 2015
…cket response header

Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Sep 1, 2015
…cket response header

Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Sep 2, 2015
…cket response header

Reviewed-by: Loic Dachary <ldachary@redhat.com>
@ghost
Copy link
Author

ghost commented Sep 2, 2015

@liewegas does this backport look good to merge ? It passed a run of the hammer rgw suite ( see http://pulpito.ceph.com/loic-2015-09-01_17:25:32-rgw-hammer-backports---basic-multi/ ).

@ghost ghost assigned liewegas and unassigned ghost Sep 6, 2015
@ghost ghost assigned yehudasa and unassigned liewegas Sep 24, 2015
@ghost
Copy link
Author

ghost commented Sep 24, 2015

@yehudasa would you mind taking a look ? It passed a run of the hammer rgw suite ( see http://pulpito.ceph.com/loic-2015-09-01_17:25:32-rgw-hammer-backports---basic-multi/ ).

yehudasa added a commit that referenced this pull request Sep 24, 2015
RGW returns requested bucket name raw in Bucket response header
@yehudasa yehudasa merged commit fe0fc4f into ceph:hammer Sep 24, 2015
@ghost ghost changed the title RGW returns requested bucket name raw in Bucket response header rgw: returns requested bucket name raw in Bucket response header Sep 30, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants