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

Don't use pointers to int64 in function calls when not necessary

Browse source
This commit is contained in:
Jeff Mitchell 2016-05-19 12:26:02 -04:00
parent 964916674e
commit 8f592f3442
2 changed files with 6 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
builtin/logical/aws/path_sts.go
View file

@ -52,7 +52,7 @@ func (b *backend) pathSTSRead(
return b.assumeRole(
req.Storage,
req.DisplayName, policyName, policyValue,
&ttl,
ttl,
)
} else {
return logical.ErrorResponse(
@ -64,7 +64,7 @@ func (b *backend) pathSTSRead(
return b.secretTokenCreate(
req.Storage,
req.DisplayName, policyName, policyValue,
&ttl,
ttl,
)
}

8
builtin/logical/aws/secret_access_keys.go
View file

@ -71,7 +71,7 @@ func genUsername(displayName, policyName, userType string) (ret string, warning
func (b *backend) secretTokenCreate(s logical.Storage,
displayName, policyName, policy string,
lifeTimeInSeconds *int64) (*logical.Response, error) {
lifeTimeInSeconds int64) (*logical.Response, error) {
STSClient, err := clientSTS(s)
if err != nil {
return logical.ErrorResponse(err.Error()), nil
@ -83,7 +83,7 @@ func (b *backend) secretTokenCreate(s logical.Storage,
&sts.GetFederationTokenInput{
Name: aws.String(username),
Policy: aws.String(policy),
DurationSeconds: lifeTimeInSeconds,
DurationSeconds: &lifeTimeInSeconds,
})
if err != nil {
@ -113,7 +113,7 @@ func (b *backend) secretTokenCreate(s logical.Storage,
func (b *backend) assumeRole(s logical.Storage,
displayName, policyName, policy string,
lifeTimeInSeconds *int64) (*logical.Response, error) {
lifeTimeInSeconds int64) (*logical.Response, error) {
STSClient, err := clientSTS(s)
if err != nil {
return logical.ErrorResponse(err.Error()), nil
@ -125,7 +125,7 @@ func (b *backend) assumeRole(s logical.Storage,
&sts.AssumeRoleInput{
RoleSessionName: aws.String(username),
RoleArn: aws.String(policy),
DurationSeconds: lifeTimeInSeconds,
DurationSeconds: &lifeTimeInSeconds,
})
if err != nil {