Merge pull request #367 from spg/fix/iam_get_role
IAM: unnecessary check for role
This commit is contained in:
		
						commit
						a3d5c771ef
					
				| @ -256,26 +256,17 @@ class IAMBackend(BaseBackend): | |||||||
| 
 | 
 | ||||||
|     def put_role_policy(self, role_name, policy_name, policy_json): |     def put_role_policy(self, role_name, policy_name, policy_json): | ||||||
|         role = self.get_role(role_name) |         role = self.get_role(role_name) | ||||||
|         if role: |  | ||||||
|         role.put_policy(policy_name, policy_json) |         role.put_policy(policy_name, policy_json) | ||||||
|         else: |  | ||||||
|             raise BotoServerError(404, 'Not Found') |  | ||||||
| 
 | 
 | ||||||
|     def get_role_policy(self, role_name, policy_name): |     def get_role_policy(self, role_name, policy_name): | ||||||
|         role = self.get_role(role_name) |         role = self.get_role(role_name) | ||||||
|         if role: |  | ||||||
|         for p, d in role.policies.items(): |         for p, d in role.policies.items(): | ||||||
|             if p == policy_name: |             if p == policy_name: | ||||||
|                 return p, d |                 return p, d | ||||||
|         else: |  | ||||||
|             raise BotoServerError(404, 'Not Found') |  | ||||||
| 
 | 
 | ||||||
|     def list_role_policies(self, role_name): |     def list_role_policies(self, role_name): | ||||||
|         role = self.get_role(role_name) |         role = self.get_role(role_name) | ||||||
|         if role: |  | ||||||
|         return role.policies.keys() |         return role.policies.keys() | ||||||
|         else: |  | ||||||
|             raise BotoServerError(404, 'Not Found') |  | ||||||
| 
 | 
 | ||||||
|     def create_instance_profile(self, name, path, role_ids): |     def create_instance_profile(self, name, path, role_ids): | ||||||
|         instance_profile_id = random_resource_id() |         instance_profile_id = random_resource_id() | ||||||
|  | |||||||
		Loading…
	
	
			
			x
			
			
		
	
		Reference in New Issue
	
	Block a user