Skip to content

Commit

Permalink
use sso role name from grant output for eks and rds access
Browse files Browse the repository at this point in the history
  • Loading branch information
JoshuaWilkes committed Nov 12, 2024
1 parent eeec10e commit bd2b3d7
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion pkg/granted/eks/eks.go
Original file line number Diff line number Diff line change
Expand Up @@ -91,7 +91,7 @@ var proxyCommand = cli.Command{
err = proxy.WaitForSSMConnectionToProxyServer(ctx, proxy.WaitForSSMConnectionToProxyServerOpts{
AWSConfig: proxy.AWSConfig{
SSOAccountID: ensuredAccess.GrantOutput.EksCluster.AccountId,
SSORoleName: ensuredAccess.Grant.Id,
SSORoleName: ensuredAccess.GrantOutput.SsoRoleName,
SSORegion: ensuredAccess.GrantOutput.SsoRegion,
SSOStartURL: ensuredAccess.GrantOutput.SsoStartUrl,
Region: ensuredAccess.GrantOutput.EksCluster.Region,
Expand Down
2 changes: 1 addition & 1 deletion pkg/granted/rds/rds.go
Original file line number Diff line number Diff line change
Expand Up @@ -92,7 +92,7 @@ var proxyCommand = cli.Command{
err = proxy.WaitForSSMConnectionToProxyServer(ctx, proxy.WaitForSSMConnectionToProxyServerOpts{
AWSConfig: proxy.AWSConfig{
SSOAccountID: ensuredAccess.GrantOutput.RdsDatabase.AccountId,
SSORoleName: ensuredAccess.Grant.Id,
SSORoleName: ensuredAccess.GrantOutput.SsoRoleName,
SSORegion: ensuredAccess.GrantOutput.SsoRegion,
SSOStartURL: ensuredAccess.GrantOutput.SsoStartUrl,
Region: ensuredAccess.GrantOutput.RdsDatabase.Region,
Expand Down

0 comments on commit bd2b3d7

Please sign in to comment.