-
Notifications
You must be signed in to change notification settings - Fork 4
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
PLTCONN-5210 assume external arn role #61
Conversation
|
1553242
to
134d4e2
Compare
this.secretAccessKey = secretAccessKey; | ||
this.sessionToken = sessionToken; | ||
this.expiration = expiration; | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could we call these something like AssumeAwsRoleRequest
and AssumeAwsRoleResponse
? Just to line up all the names.
lib/connector-handler.ts
Outdated
assumeAwsRole(externalRoleInput: ExternalRoleInput): Promise<CredentialResponse>; | ||
} | ||
export class ExternalRoleInput { | ||
externalArnRole: string; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could we call this roleArn
to line up with AssumeRole API call we'll eventually make?
7deb358
to
874ef2f
Compare
567c156
to
d682271
Compare
Description
What is the intent of this change and why is it being made?
How Has This Been Tested?
What testing have you done to verify this change?