Skip to content

Commit

Permalink
Merge pull request #10664: jewel: Bug when using port 443s in rgw.
Browse files Browse the repository at this point in the history
Reviewed-by: Loic Dachary <ldachary@redhat.com>
  • Loading branch information
Loic Dachary committed Aug 12, 2016
2 parents bb54484 + 22c9df1 commit 7fb3d17
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 7fb3d17

Please sign in to comment.