Skip to content

Commit

Permalink
rgw: fix for zonegroup redirect url
Browse files Browse the repository at this point in the history
local dest_url variable was shadowing the one in the enclosing scope, so
the changes were not applied and no Location header was written on redirect

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

Signed-off-by: Casey Bodley <cbodley@redhat.com>
  • Loading branch information
cbodley committed Apr 4, 2017
1 parent 92f63c6 commit 542e188
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/rgw/rgw_rest.cc
Original file line number Diff line number Diff line change
Expand Up @@ -775,7 +775,7 @@ void abort_early(struct req_state *s, RGWOp *op, int err_no,
if (!s->redirect.empty()) {
dest_uri = s->redirect;
} else if (!s->zonegroup_endpoint.empty()) {
string dest_uri = s->zonegroup_endpoint;
dest_uri = s->zonegroup_endpoint;
/*
* reqest_uri is always start with slash, so we need to remove
* the unnecessary slash at the end of dest_uri.
Expand Down

0 comments on commit 542e188

Please sign in to comment.