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
kraken: rgw: Use decoded URI when verifying TempURL #12986
Conversation
@Werkov I think this needs to be targeted to master. After it is merged to master it can be backported to kraken, etc. |
@Werkov After the master PR is merged, please re-do this as a cherry-pick using |
@Werkov The master commit has been merged. Please go ahead as described in the previous comment. |
Instead of calliing url_decode directly, we reuse s->decoded_uri that is initialized in RGWREST::preprocess(). Fixes: http://tracker.ceph.com/issues/18590 Signed-off-by: Michal Koutný <mkoutny@suse.com> (cherry picked from commit 4e1318f)
2a95399
to
bfc058d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
@smithfarm: this PR looks like a Kraken backport. Should we change the title and update the |
@rzarzynski Thanks; done! |
This PR passed an RGW suite at http://tracker.ceph.com/issues/19009#note-18 |
http://tracker.ceph.com/issues/18627