Skip to content

Commit

Permalink
Merge pull request #149 from Cox-Automotive/develop
Browse files Browse the repository at this point in the history
Develop
  • Loading branch information
amagana3 committed May 20, 2020
2 parents aae8936 + 92c7a93 commit d877af8
Show file tree
Hide file tree
Showing 6 changed files with 9 additions and 7 deletions.
3 changes: 2 additions & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -134,7 +134,7 @@ Arguments:
* `-a [account]` The ALKS account to use, be sure to wrap in quotes
* `-r [role]` The ALKS role to use, be sure to wrap in quotes
* `-i` Specifies you wish to work as an IAM/Admin user
* `-o [output]` Output format. Supports: `env`, `json`, `docker`, `creds`, `idea`, `export`, `set`, `powershell`
* `-o [output]` Output format. Supports: `env`, `json`, `docker`, `creds`, `idea`, `export`, `set`, `powershell`, `aws`, `fishshell`, `terraformenv`, `terraformarg`
* `-n` If output is set to creds, use this named profile (defaults to default)
* `-N` Forces a new session to be generated
* `-d` Uses your default account from `alks developer configure`
Expand All @@ -154,6 +154,7 @@ Output values:
Arguments:

* `-p [password]` Your password
* `-u [url]` Print the URL
* `-a [account]` The ALKS account to use, be sure to wrap in quotes
* `-r [role]` The ALKS role to use, be sure to wrap in quotes
* `-i` Specifies you wish to work as an IAM/Admin user
Expand Down
5 changes: 3 additions & 2 deletions changelog.txt
Original file line number Diff line number Diff line change
@@ -1,9 +1,10 @@
★ Release Notes: 2020-04-15
★ Release Notes: 2020-05-20
≡≡≡≡≡≡≡≡≡≡≡≡≡≡≡≡≡≡≡≡≡≡≡≡≡≡≡≡≡

Thanks for upgrading to the latest version of the ALKS CLI!

→ Converts a dependency to a tarball to fix breakages on install
→ Updated README.md to include `-o` and `-u` documentation
→ Normalized the "expires" field when working with sessions

→ Have feedback? https://github.com/Cox-Automotive/ALKS-CLI/issues

Expand Down
2 changes: 1 addition & 1 deletion lib/keys.js
Original file line number Diff line number Diff line change
Expand Up @@ -130,7 +130,7 @@ exports.addKey = function(accessKey, secretKey, sessionToken, alksAccount, alksR
alksAccount: encrypt(alksAccount, enc),
alksRole: encrypt(alksRole, enc),
isIAM: isIAM,
expires: expires.getTime()
expires: expires
});

db.save(function(err, data){
Expand Down
2 changes: 1 addition & 1 deletion lib/sessions.js
Original file line number Diff line number Diff line change
Expand Up @@ -93,7 +93,7 @@ exports.getSessionKey = function(program, logger, alksAccount, alksRole, iamOnly
// store session data in DB
utils.log(program, logger, 'storing key: ' + JSON.stringify(key));
keys.addKey(key.accessKey, key.secretKey, key.sessionToken,
key.alksAccount, key.alksRole, key.expires.toDate(), auth, false);
key.alksAccount, key.alksRole, key.expires, auth, false);
}
else if(err.message.indexOf('please check API URL') !== -1){
err = new Error(utils.getBadAccountMessage());
Expand Down
2 changes: 1 addition & 1 deletion npm-shrinkwrap.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "alks",
"version": "3.0.7",
"version": "3.0.8",
"description": "CLI for working with ALKS",
"main": "bin/alks",
"scripts": {
Expand Down

0 comments on commit d877af8

Please sign in to comment.