[aws/sts_provisioner.go] Don't favor on default_region

Default region should only be used to fill in a region when
awsConfig.Region doesn't exist (and set it accordingly, but not be the
sole determinant on if there is a difference.  It should defer to what
is already does in the `else if` later, where it sets it if it isn't
blank, but doesn't error.

Right now, it will error if `default region` is set in the 1Password
item, but in reality, it should only do that if `region` (aka:
`hasRegularRegion`) is set.

This updates this logic to reflect that.

Author: NickLaMuro

plugins/aws/sts_provisioner.go

@@ -250,8 +250,8 @@ Learn how to add an OTP field to your item:
 https://developer.1password.com/docs/cli/shell-plugins/aws/#optional-set-up-multi-factor-authentication`, awsConfig.MfaSerial)
 	}
 
-	if hasRegion && awsConfig.Region != "" && region != awsConfig.Region {
-		return fmt.Errorf("your local AWS configuration (config file or environment variable) has a different default region than the one specified in 1Password")
+	if hasRegularRegion && awsConfig.Region != "" && region != awsConfig.Region {
+		return fmt.Errorf("your local AWS configuration (config file or environment variable) has a different region than the one specified in 1Password")
 	} else if awsConfig.Region == "" {
 		awsConfig.Region = region
 	}