]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
rgw: fix for zonegroup redirect url
authorCasey Bodley <cbodley@redhat.com>
Tue, 4 Apr 2017 14:42:44 +0000 (10:42 -0400)
committerNathan Cutler <ncutler@suse.com>
Thu, 6 Jul 2017 18:08:18 +0000 (20:08 +0200)
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>
(cherry picked from commit 542e188a40f0495720b48308372366951ae41e62)

src/rgw/rgw_rest.cc

index c3e119c874586314d078d64770763912939a0921..115f4c99718f53212607e8898d4c1034ec2d5b28 100644 (file)
@@ -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.