Merge pull request #82 from trussworks/cg_base_profile_mfa

Re-add the mfa device to the base profile.
trussworks · Jul 21, 2020 · 80b656a · 80b656a
2 parents 67a0f96 + 8d68ae1
commit 80b656a
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/cmd/setup.go b/cmd/setup.go
@@ -415,8 +415,9 @@ func (sc *SetupConfig) UpdateAWSConfigFile() error {
 	}
 
 	sc.BaseProfile = &vault.ProfileSection{
-		Name:   sc.BaseProfileName,
-		Region: sc.Region,
+		Name:      sc.BaseProfileName,
+		Region:    sc.Region,
+		MfaSerial: sc.MFASerial,
 	}
 
 	// Add the base profile

diff --git a/cmd/setup_test.go b/cmd/setup_test.go
@@ -91,7 +91,7 @@ output=json
 
 	testBaseSection, ok := config.ProfileSection("test-id-base")
 	suite.True(ok)
-	suite.Equal(len(testBaseSection.MfaSerial), 0)
+	suite.Equal(testBaseSection.MfaSerial, mfaSerial)
 	suite.Equal(testBaseSection.Region, "us-west-2")
 	// suite.Equal(testBaseSection.Output, "json")