RepoMirrors/ceph
1
0
Fork 0
mirror of https://github.com/ceph/ceph synced 2025-02-24 19:47:44 +00:00
Code Issues Packages Projects Releases Wiki Activity

rgw: fix for empty query string in beast frontend

Browse Source 
when the target does not contain a ?, the QUERY_STRING was being set to
the same value as REQUEST_URI. this QUERY_STRING is included in the
signature, and caused SignatureDoesNotMatch failures

Fixes: http://tracker.ceph.com/issues/22797

Signed-off-by: Casey Bodley <cbodley@redhat.com>
This commit is contained in:
Casey Bodley 2018-01-25 12:09:20 -05:00
parent 52bf90bb10
commit ab9e79684a
1 changed files with 5 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
src/rgw/rgw_asio_client.cc
View File

@ -65,11 +65,12 @@ void ClientIO::init_env(CephContext *cct)
// split uri from query
auto url = request.target();
auto pos = url.find('?');
auto query = url.substr(pos + 1);
url = url.substr(0, pos);
if (pos != url.npos) {
auto query = url.substr(pos + 1);
env.set("QUERY_STRING", query.to_string());
url = url.substr(0, pos);
}
env.set("REQUEST_URI", url.to_string());
env.set("QUERY_STRING", query.to_string());
env.set("SCRIPT_URI", url.to_string()); /* FIXME */
char port_buf[16];