Skip to content
This repository was archived by the owner on Jun 13, 2023. It is now read-only.

Commit

Permalink
Revert "create prefix for assumeRoleArn (uswitch#459)"
Browse files Browse the repository at this point in the history
This reverts commit 8103483.
  • Loading branch information
GenPage authored Jun 12, 2023
1 parent b795246 commit 06e3b58
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 10 deletions.
2 changes: 1 addition & 1 deletion cmd/kiam/agent.go
Original file line number Diff line number Diff line change
Expand Up @@ -89,7 +89,7 @@ func (opts *agentCommand) run() error {
b := kiamserver.NewKiamGatewayBuilder().WithAddress(opts.serverAddress).WithKeepAlive(opts.keepaliveParams)
_, err := b.WithTLS(opts.certificatePath, opts.keyPath, opts.caPath)
if err != nil {
log.Errorf("error configuring TLS: %s", err.Error())
log.Errorf("error configuring TLS: ", err.Error())
return err
}

Expand Down
2 changes: 2 additions & 0 deletions pkg/aws/sts/cache.go
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,8 @@ import (
)

type credentialsCache struct {
arnResolver ARNResolver
baseARN string
cache *cache.Cache
expiring chan *CachedCredentials
sessionName string
Expand Down
10 changes: 1 addition & 9 deletions pkg/server/server_builder.go
Original file line number Diff line number Diff line change
Expand Up @@ -61,15 +61,7 @@ func (b *KiamServerBuilder) WithAWSSTSGateway() (*KiamServerBuilder, error) {
if err != nil {
return nil, err
}
arnResolver, err := newRoleARNResolver(b.config)
if err != nil {
return nil, err
}
assumeRoleARN, err := arnResolver.Resolve(b.config.AssumeRoleArn)
if err != nil {
return nil, err
}
cfg.WithCredentialsFromAssumedRole(sts.NewSTSCredentialsProvider(), assumeRoleARN.ARN)
cfg.WithCredentialsFromAssumedRole(sts.NewSTSCredentialsProvider(), b.config.AssumeRoleArn)
stsGateway, err := sts.DefaultGateway(cfg.Config())
if err != nil {
return nil, err
Expand Down

0 comments on commit 06e3b58

Please sign in to comment.