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

Conversation

Projects
None yet
3 participants
@ghost

ghost commented Aug 30, 2015

rgw: url encode exposed bucket
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

Merge pull request #5715: RGW returns requested bucket name raw in Bu…
…cket response header

Reviewed-by: Loic Dachary <ldachary@redhat.com>

ghost pushed a commit that referenced this pull request Aug 30, 2015

Merge pull request #5715: RGW returns requested bucket name raw in Bu…
…cket response header

Reviewed-by: Loic Dachary <ldachary@redhat.com>

ghost pushed a commit that referenced this pull request Aug 30, 2015

Merge pull request #5715: RGW returns requested bucket name raw in Bu…
…cket response header

Reviewed-by: Loic Dachary <ldachary@redhat.com>

ghost pushed a commit that referenced this pull request Aug 31, 2015

Merge pull request #5715: RGW returns requested bucket name raw in Bu…
…cket response header

Reviewed-by: Loic Dachary <ldachary@redhat.com>

ghost pushed a commit that referenced this pull request Aug 31, 2015

Merge pull request #5715: RGW returns requested bucket name raw in Bu…
…cket response header

Reviewed-by: Loic Dachary <ldachary@redhat.com>

ghost pushed a commit that referenced this pull request Aug 31, 2015

Merge pull request #5715: RGW returns requested bucket name raw in Bu…
…cket response header

Reviewed-by: Loic Dachary <ldachary@redhat.com>

ghost pushed a commit that referenced this pull request Sep 1, 2015

Merge pull request #5715: RGW returns requested bucket name raw in Bu…
…cket response header

Reviewed-by: Loic Dachary <ldachary@redhat.com>

ghost pushed a commit that referenced this pull request Sep 2, 2015

Merge pull request #5715: RGW returns requested bucket name raw in Bu…
…cket response header

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

This comment has been minimized.

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

This comment has been minimized.

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

Merge pull request #5715 from dachary/wip-12592-hammer
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 from RGW returns requested bucket name raw in Bucket response header to 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