Merge pull request #415 from patleb/iam-list-access_keys

correction in iam/list_access_keys parser: Username should be UserName
fog · Nov 22, 2017 · 710ed63 · 710ed63
2 parents 1caf487 + e869218
commit 710ed63
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 3 deletions.
diff --git a/lib/fog/aws/models/iam/access_keys.rb b/lib/fog/aws/models/iam/access_keys.rb
@@ -13,8 +13,6 @@ def initialize(attributes = {})
 
         def all
           data = service.list_access_keys('UserName'=> @username).body['AccessKeys']
-          # AWS response doesn't contain the UserName, this injects it
-          data.each {|access_key| access_key['UserName'] = @username }
           load(data)
         end
 

diff --git a/lib/fog/aws/parsers/iam/list_access_keys.rb b/lib/fog/aws/parsers/iam/list_access_keys.rb
@@ -10,7 +10,7 @@ def reset
 
           def end_element(name)
             case name
-            when 'AccessKeyId', 'Status', 'Username'
+            when 'AccessKeyId', 'Status', 'UserName'
               @access_key[name] = value
             when 'member'
               @response['AccessKeys'] << @access_key