Skip to content

Commit

Permalink
rgw: Fix for using port 443 with pre-signed urls.
Browse files Browse the repository at this point in the history
Fixes http://tracker.ceph.com/issues/16548

Signed-off-by: Pritha Srivastava <prsrivas@redhat.com>
(cherry picked from commit 44b9ed3)
  • Loading branch information
pritha-srivastava authored and ldachary committed Aug 10, 2016
1 parent c587e9e commit 22c9df1
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/rgw/rgw_rest_s3.cc
Original file line number Diff line number Diff line change
Expand Up @@ -3484,7 +3484,7 @@ int RGW_Auth_S3::authorize_v4(RGWRados *store, struct req_state *s)
}
string token_value = string(t);
if (using_qs && (token == "host")) {
if (!port.empty() && port != "80") {
if (!port.empty() && port != "80" && port != "0") {
token_value = token_value + ":" + port;
} else if (!secure_port.empty() && secure_port != "443") {
token_value = token_value + ":" + secure_port;
Expand Down

0 comments on commit 22c9df1

Please sign in to comment.