From 859842af1fd083ffe7e8c88fdba238886c03c4e3 Mon Sep 17 00:00:00 2001 From: Bryant Biggs Date: Tue, 26 Aug 2025 09:17:37 -0500 Subject: [PATCH] fix: Correct `iam-group` variable name mis-spelling --- docs/UPGRADE-6.0.md | 2 +- modules/iam-group/README.md | 2 +- modules/iam-group/main.tf | 2 +- modules/iam-group/variables.tf | 2 +- wrappers/iam-group/main.tf | 2 +- 5 files changed, 5 insertions(+), 5 deletions(-) diff --git a/docs/UPGRADE-6.0.md b/docs/UPGRADE-6.0.md index ed390526..ce3b9cf4 100644 --- a/docs/UPGRADE-6.0.md +++ b/docs/UPGRADE-6.0.md @@ -134,7 +134,7 @@ stateDiagram - None - `iam-group` - `create_group` -> `create` - - `group_users` -> `group` + - `group_users` -> `users` - `custom_group_policy_arns` -> `policies` - `attach_iam_self_management_policy` -> `create_policy` - `iam_self_management_policy_name_prefix` -> `policy_name_prefix` diff --git a/modules/iam-group/README.md b/modules/iam-group/README.md index f05a3efb..6439eba0 100644 --- a/modules/iam-group/README.md +++ b/modules/iam-group/README.md @@ -72,7 +72,7 @@ No modules. |------|-------------|------|---------|:--------:| | [create](#input\_create) | Controls if resources should be created (affects all resources) | `bool` | `true` | no | | [create\_policy](#input\_create\_policy) | Whether to create IAM policy for IAM group | `bool` | `true` | no | -| [enable\_mfa\_enforcment](#input\_enable\_mfa\_enforcment) | Determines whether permissions are added to the policy which requires the groups IAM users to use MFA | `bool` | `true` | no | +| [enable\_mfa\_enforcement](#input\_enable\_mfa\_enforcement) | Determines whether permissions are added to the policy which requires the groups IAM users to use MFA | `bool` | `true` | no | | [enable\_self\_management\_permissions](#input\_enable\_self\_management\_permissions) | Determines whether permissions are added to the policy which allow the groups IAM users to manage their credentials and MFA | `bool` | `true` | no | | [name](#input\_name) | The group's name. The name must consist of upper and lowercase alphanumeric characters with no spaces. You can also include any of the following characters: `=,.@-_.` | `string` | `""` | no | | [path](#input\_path) | Path in which to create the group | `string` | `null` | no | diff --git a/modules/iam-group/main.tf b/modules/iam-group/main.tf index cb00e18a..baf48d7a 100644 --- a/modules/iam-group/main.tf +++ b/modules/iam-group/main.tf @@ -171,7 +171,7 @@ data "aws_iam_policy_document" "this" { } dynamic "statement" { - for_each = var.enable_self_management_permissions && var.enable_mfa_enforcment ? [1] : [] + for_each = var.enable_self_management_permissions && var.enable_mfa_enforcement ? [1] : [] content { sid = "DenyAllExceptListedIfNoMFA" diff --git a/modules/iam-group/variables.tf b/modules/iam-group/variables.tf index 87b3bdd7..ea42cf0a 100644 --- a/modules/iam-group/variables.tf +++ b/modules/iam-group/variables.tf @@ -48,7 +48,7 @@ variable "enable_self_management_permissions" { default = true } -variable "enable_mfa_enforcment" { +variable "enable_mfa_enforcement" { description = "Determines whether permissions are added to the policy which requires the groups IAM users to use MFA" type = bool default = true diff --git a/wrappers/iam-group/main.tf b/wrappers/iam-group/main.tf index 1a0bd85f..69434be6 100644 --- a/wrappers/iam-group/main.tf +++ b/wrappers/iam-group/main.tf @@ -5,7 +5,7 @@ module "wrapper" { create = try(each.value.create, var.defaults.create, true) create_policy = try(each.value.create_policy, var.defaults.create_policy, true) - enable_mfa_enforcment = try(each.value.enable_mfa_enforcment, var.defaults.enable_mfa_enforcment, true) + enable_mfa_enforcement = try(each.value.enable_mfa_enforcement, var.defaults.enable_mfa_enforcement, true) enable_self_management_permissions = try(each.value.enable_self_management_permissions, var.defaults.enable_self_management_permissions, true) name = try(each.value.name, var.defaults.name, "") path = try(each.value.path, var.defaults.path, null)