Merge pull request #1717 from rcrews/fix-qs-ex1-warnings

Fix QuickStart example #1 warnings

Author: awood45

gems/aws-sdk-core/CHANGELOG.md

@@ -6,6 +6,8 @@ Unreleased Changes
 
 * Feature - Updated Aws::STS::Client with the latest API changes.
 
+* Issue - Fix Ruby warnings: Shadowed local variables "parts" and "headers"
+
 3.16.0 (2018-02-20)
 ------------------
 

gems/aws-sdk-core/lib/aws-sdk-core/rest/request/endpoint.rb

@@ -49,11 +49,11 @@ def param_name(placeholder)
 
         def apply_querystring_params(uri, params)
           # collect params that are supposed to be part of the query string
-          parts = @rules.shape.members.inject([]) do |parts, (member_name, member_ref)|
+          parts = @rules.shape.members.inject([]) do |prts, (member_name, member_ref)|
             if member_ref.location == 'querystring' && !params[member_name].nil?
-              parts << [member_ref, params[member_name]]
+              prts << [member_ref, params[member_name]]
             end
-            parts
+            prts
           end
           querystring = QuerystringBuilder.new.build(parts)
           querystring = [@query_prefix, querystring == '' ? nil : querystring].compact.join('&')

gems/aws-sigv4/CHANGELOG.md

@@ -1,6 +1,11 @@
 Unreleased Changes
 ------------------
 
+1.0.2 (2018-02-21)
+------------------
+
+* Issue - Fix Ruby warning: shadowed local variable "headers"
+
 1.0.2 (2017-08-31)
 ------------------
 

gems/aws-sigv4/lib/aws-sigv4/signer.rb

@@ -438,11 +438,11 @@ def signed_headers(headers)
       end
 
       def canonical_headers(headers)
-        headers = headers.inject([]) do |headers, (k,v)|
+        headers = headers.inject([]) do |hdrs, (k,v)|
           if @unsigned_headers.include?(k)
-            headers
+            hdrs
           else
-            headers << [k,v]
+            hdrs << [k,v]
           end
         end
         headers = headers.sort_by(&:first)