From 3e33b71cfddc88e653abb8e4cd33cd20c2719483 Mon Sep 17 00:00:00 2001
From: Joe Khoobyar <joekhoobyar@users.noreply.github.com>
Date: Tue, 10 Dec 2024 12:07:40 -0500
Subject: [PATCH] Fix inability to use an account Id with leading zeroes
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

You can't make the account ID a number, because then leading zeroes will get stripped and you'll get this error:

Assume the account Id is "012345678901"
╷
│ Error: "account_id" doesn't look like AWS Account ID (exactly 12 digits): "12345678901"
│
│   with module.securityhub-member[0].aws_securityhub_member.this["12345678901"],
│   on .terraform/modules/securityhub-member/modules/organizations_member/main.tf line 7, in resource "aws_securityhub_member" "this":
│    7:   account_id = each.value.account_id
---
 modules/organizations_member/variables.tf | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/modules/organizations_member/variables.tf b/modules/organizations_member/variables.tf
index 061c9f2..8505876 100644
--- a/modules/organizations_member/variables.tf
+++ b/modules/organizations_member/variables.tf
@@ -4,12 +4,12 @@
 variable "member_config" {
   description = <<EOF
   Specifies the member account configuration:
-  `account_id`                 - The 13 digit ID number of the member account. Example: `123456789012`.
+  `account_id`                 - The 12 digit ID number of the member account. Example: `123456789012`.
   `email`                      - Email address to send the invite for member account. Defaults to `null`.
   `invite`                     - Whether to invite the account to SecurityHub as a member. Defaults to `false`. To detect if an invitation needs to be (re-)sent, the Terraform state value is true based on a `member_status` of `Disabled` | `Enabled` |  `Invited` |  EmailVerificationInProgress.
   EOF
   type = list(object({
-    account_id = number
+    account_id = string
     email      = string
     invite     = bool
   }))