Skip to content

Commit

Permalink
Merge pull request #443 from jackjen/fix-eval-refresh
Browse files Browse the repository at this point in the history
Fix eval refresh not finding role arn
  • Loading branch information
synfinatic authored Nov 10, 2022
2 parents 3c9d510 + 38e21fa commit d071510
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion cmd/aws-sso/eval_cmd.go
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,7 @@ func (cc *EvalCmd) Run(ctx *RunContext) error {

// refreshing?
if ctx.Cli.Eval.Refresh {
if ctx.Cli.Eval.EnvArn != "" {
if ctx.Cli.Eval.EnvArn == "" {
return fmt.Errorf("Unable to determine current IAM role")
}
accountid, role, err = utils.ParseRoleARN(ctx.Cli.Eval.EnvArn)
Expand Down

0 comments on commit d071510

Please sign in to comment.