Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
Co-authored-by: Herman Slatman <[email protected]>
  • Loading branch information
leonweecs and hslatman committed Sep 12, 2024
1 parent 44ba5ce commit 4d058a5
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions cas/vaultcas/auth/aws/aws.go
Original file line number Diff line number Diff line change
Expand Up @@ -65,14 +65,14 @@ func NewAwsAuthMethod(mountPath string, options json.RawMessage) (*aws.AWSAuth,
case "":
// no-op
default:
return nil, fmt.Errorf("unknown SignatureType type: %s, valid options are 'pkcs7', 'identity' and 'rsa2048'", opts.SignatureType)
return nil, fmt.Errorf("unknown SignatureType type %q; valid options are 'pkcs7', 'identity' and 'rsa2048'", opts.SignatureType)
}

if opts.Nonce != "" {
loginOptions = append(loginOptions, aws.WithNonce(opts.Nonce))
}
default:
return nil, fmt.Errorf("unknown awsAuthType: %s, valid options are 'iam' and 'ec2'", opts.AwsAuthType)
return nil, fmt.Errorf("unknown awsAuthType %q; valid options are 'iam' and 'ec2'", opts.AwsAuthType)
}

awsAuth, err = aws.NewAWSAuth(loginOptions...)
Expand Down

0 comments on commit 4d058a5

Please sign in to comment.