Skip to content

Commit

Permalink
[aws/sts_provisioner.go] Remove hasRegion
Browse files Browse the repository at this point in the history
Now not used by the error check below, so removing.
  • Loading branch information
NickLaMuro committed Aug 6, 2024
1 parent cd33c6b commit 2d45c84
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions plugins/aws/sts_provisioner.go
Original file line number Diff line number Diff line change
Expand Up @@ -227,8 +227,6 @@ func resolveLocalAnd1PasswordConfigurations(itemFields map[sdk.FieldName]string,
region = defaultRegion
}

hasRegion := hasDefaultRegion || hasRegularRegion

// only 1Password OTPs are supported
if awsConfig.MfaToken != "" || awsConfig.MfaProcess != "" || awsConfig.MfaPromptMethod != "" {
return fmt.Errorf("only 1Password-backed OTP authentication is supported by the MFA worklfow of the AWS shell plugin")
Expand Down

0 comments on commit 2d45c84

Please sign in to comment.