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

ex-ML #15586

Merged
merged 1 commit into from Jun 13, 2017

Conversation

Projects
None yet
4 participants
@adamemerson
Contributor

adamemerson commented Jun 8, 2017

Remove spurious XML header from the JSON GetBucketPolicy response.

@adamemerson adamemerson requested a review from oritwas Jun 8, 2017

@oritwas

oritwas approved these changes Jun 9, 2017

@oritwas

This comment has been minimized.

Contributor

oritwas commented Jun 9, 2017

@adamemerson , does it have a related tracker issue?

@tchaikov tchaikov added the rgw label Jun 9, 2017

@adamemerson

This comment has been minimized.

Contributor

adamemerson commented Jun 9, 2017

@oritwas No. Someone just emailed me about it

@joke-lee

This comment has been minimized.

Contributor

joke-lee commented Jun 12, 2017

@oritwas

This comment has been minimized.

Contributor

oritwas commented Jun 13, 2017

@adamemerson , can you update the commit with tracker issue link?

rgw: Remove spurious XML header for GetBucketPolicy
Fixes: http://tracker.ceph.com/issues/20247
Signed-off-by: Adam C. Emerson <aemerson@redhat.com>

@adamemerson adamemerson merged commit 6eb7822 into ceph:master Jun 13, 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

adamemerson added a commit that referenced this pull request Jun 13, 2017

Merge pull request #15586 from adamemerson/wip-ex-ML
rgw: Remove spurious XML header for GetBucketPolicy

Reviewed-by: Orit Wasserman <owasserm@redhat.com>

@adamemerson adamemerson deleted the adamemerson:wip-ex-ML branch Jun 13, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment