Merge pull request #327 from rodrigoapereira/fix_host_canonical_headers

fix host header with another port on s3
fog · Dec 6, 2016 · 0a247f8 · 0a247f8
2 parents 9640d7a + c3131a4
commit 0a247f8
Showing 1 changed file with 1 addition and 0 deletions.
diff --git a/lib/fog/aws/storage.rb b/lib/fog/aws/storage.rb
@@ -190,6 +190,7 @@ def v4_signed_params_for_url(params, expires)
 
           params = request_params(params)
           params[:headers][:host] = params[:host]
+          params[:headers][:host] += ":#{params[:port]}" if params.fetch(:port, nil)
 
           signature_query_params = @signer.signature_parameters(params, now, "UNSIGNED-PAYLOAD")
           params[:query] = (params[:query] || {}).merge(signature_query_params)