luxolus/open-vault
2
0
Fork 0
Code Issues 1 Pull requests Releases Activity

Remove the sanity check which is not proving to be useful

Browse source
This commit is contained in:
vishalnayak 2016-10-27 19:11:26 -04:00
parent 65b65eba30
commit 65f0ce8ca3
1 changed files with 0 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
builtin/credential/aws-ec2/path_role.go
View file

@ -207,11 +207,6 @@ func (b *backend) nonLockedAWSRole(s logical.Storage, roleName string) (*awsRole
// Check if the value held by role ARN field is actually an instance profile ARN
if result.BoundIamRoleARN != "" && strings.Contains(result.BoundIamRoleARN, ":instance-profile/") {
// For sanity
if result.BoundIamInstanceProfileARN != "" {
return nil, fmt.Errorf("bound_iam_role_arn contains instance profile ARN and bound_iam_instance_profile_arn is non empty")
}
// If yes, move it to the correct field
result.BoundIamInstanceProfileARN = result.BoundIamRoleARN