Only need to skip MFA if it is present

99designs · Jan 8, 2020 · b5e0cab · b5e0cab
1 parent 8e44923
commit b5e0cab
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/vault/vault.go b/vault/vault.go
@@ -147,7 +147,7 @@ func (c *CredentialLoader) ProviderWithChainedMfa(profileName string, isChained
 		sourceCredProvider = NewMasterCredentialsProvider(c.Keyring, config.ProfileName)
 	} else if config.SourceProfile != "" {
 		sourceCredProvider, err = c.ProviderWithChainedMfa(config.SourceProfile, true, config.MfaSerial)
-		if err == nil {
+		if err == nil && config.MfaSerial != "" {
 			skipMfaBecauseSourceProfileHasItCovered = true
 			config.MfaSerial = ""
 		}