From 2b203a5d7b15c5c0f69462dd9bb668efeb9f25fc Mon Sep 17 00:00:00 2001 From: geemus Date: Fri, 14 May 2010 12:01:38 -0700 Subject: [PATCH] [s3] fix s3 signing with query params, accidently made it too lax while working on versioning, etc --- lib/fog/aws/s3.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/fog/aws/s3.rb b/lib/fog/aws/s3.rb index 2bb7967c67..47446e269f 100644 --- a/lib/fog/aws/s3.rb +++ b/lib/fog/aws/s3.rb @@ -203,7 +203,7 @@ def signature(params) canonical_resource << "#{CGI.escape(subdomain).downcase}/" end canonical_resource << "#{params[:path]}" - if params[:query] && !params[:query].empty? + if ['acl', 'location', 'logging', 'requestPayment', 'torrent', 'versioning'].include?(params[:query]) canonical_resource << "?#{params[:query]}" end string_to_sign << "#{canonical_resource}"