fix(aws): Remove account validation

erezrokah · erezrokah · commit 56232887db3d · 2023-01-02T10:39:04.000+02:00
diff --git a/plugins/source/aws/client/spec.go b/plugins/source/aws/client/spec.go
@@ -63,9 +63,6 @@ func (s *Spec) Validate() error {
 		if s.Organization.ChildAccountRoleName == "" {
 			return fmt.Errorf("member_role_name is required when using org configuration")
 		}
-		if err := validateAccounts(s.Organization.SkipMemberAccounts); err != nil {
-			return fmt.Errorf("invalid skip_member_accounts: %w", err)
-		}
 		if err := validateOUs(s.Organization.OrganizationUnits); err != nil {
 			return fmt.Errorf("invalid organization_units: %w", err)
 		}
@@ -77,16 +74,6 @@ func (s *Spec) Validate() error {
 	return nil
 }
 
-func validateAccounts(accounts []string) error {
-	r := regexp.MustCompile(`^(\d{12})$`)
-	for _, account := range accounts {
-		if !r.MatchString(account) {
-			return fmt.Errorf("invalid account id: %s (should be 12 digits)", account)
-		}
-	}
-	return nil
-}
-
 func validateOUs(ous []string) error {
 	r := regexp.MustCompile(`^((ou\-[0-9a-z]{4,32}\-[a-z0-9]{8,32})|(r\-[0-9a-z]{4,32}))$`)
 	for _, ou := range ous {
diff --git a/plugins/source/aws/client/spec_test.go b/plugins/source/aws/client/spec_test.go
@@ -9,10 +9,11 @@ func TestSpecValidate(t *testing.T) {
 		wantErr bool
 	}{
 		{
-			name: "valid account",
+			name: "valid accounts",
 			spec: &Spec{
 				Accounts: []Account{
 					{ID: "123456789012"},
+					{ID: "cq-playground"},
 				},
 			},
 			wantErr: false,
