Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add aws_security_key to credentials file for ansible compatibility #12

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 4 additions & 2 deletions background/script.js
Original file line number Diff line number Diff line change
Expand Up @@ -138,7 +138,8 @@ function extractPrincipalPlusRoleAndAssumeRole(samlattribute, SAMLAssertion) {
var docContent = "[default] \n" +
"aws_access_key_id = " + data.Credentials.AccessKeyId + " \n" +
"aws_secret_access_key = " + data.Credentials.SecretAccessKey + " \n" +
"aws_session_token = " + data.Credentials.SessionToken;
"aws_session_token = " + data.Credentials.SessionToken + " \n" +
"aws_security_token = " + data.Credentials.SessionToken;

// If there are no Role ARNs configured in the options panel, continue to create credentials file
// Otherwise, extend docContent with a profile for each specified ARN in the options panel
Expand Down Expand Up @@ -174,7 +175,8 @@ function assumeAdditionalRole(profileList, index, AccessKeyId, SecretAccessKey,
"[" + profileList[index] + "] \n" +
"aws_access_key_id = " + data.Credentials.AccessKeyId + " \n" +
"aws_secret_access_key = " + data.Credentials.SecretAccessKey + " \n" +
"aws_session_token = " + data.Credentials.SessionToken;
"aws_session_token = " + data.Credentials.SessionToken + " \n" +
"aws_security_token = " + data.Credentials.SessionToken;
}
// If there are more profiles/roles in the RoleArns dict, do another call of assumeAdditionalRole to extend the docContent with another profile
// Otherwise, this is the last profile/role in the RoleArns dict. Proceed to creating the credentials file
Expand Down