Skip to content

Fix typo in variable name nat_rule_name_shortned - #212

Merged
athiththan11 merged 1 commit into
wso2:mainfrom
kavix:feature/fix-nat-rule-name-typo
Jun 17, 2026
Merged

athiththan11 merged 1 commit into
wso2:mainfrom
kavix:feature/fix-nat-rule-name-typo

Conversation

@kavix

@kavix kavix commented Jun 13, 2026

Copy link
Copy Markdown
Contributor

Purpose

Corrects a typo in the variable name nat_rule_name_shortned used in the Azure firewall modules.
Resolves #157

Goals

Ensure correct spelling (nat_rule_name_shortened) of the variable in variable definitions and its resource implementations.

Approach

Updated references to the variable nat_rule_name_shortned to nat_rule_name_shortened in the following files:

  • modules/azurerm/Firewall-LB-DNAT-Rule-Collection/variables.tf
  • modules/azurerm/Firewall-LB-DNAT-Rule-Collection/firewall_dnat_rules.tf
  • modules/azurerm/Firewall-Multiple-PIP/variables.tf
  • modules/azurerm/Firewall-Multiple-PIP/nat_rules.tf

Ran terraform fmt -recursive to ensure standard HCL formatting is preserved.

User stories

N/A

Release note

Corrected the typo in variable name nat_rule_name_shortned to nat_rule_name_shortened in azurerm Firewall modules.

Documentation

N/A - This is a bug fix for an internal variable naming typo.

Training

N/A

Certification

N/A - This change has no impact on certification exams.

Marketing

N/A

Automation tests

  • Unit tests
    N/A
  • Integration tests
    Validated configurations successfully via local terraform init and terraform validate on the affected modules.

Security checks

Samples

N/A

Related PRs

N/A

Migrations (if applicable)

Users of the Firewall-LB-DNAT-Rule-Collection and Firewall-Multiple-PIP modules who define the nat_rule_name_shortned attribute in their dynamic_nat_rules configuration block will need to update the key name to nat_rule_name_shortened when upgrading.

Test environment

  • OS: macOS
  • Terraform version: v1.15.5

Learning

N/A

@CLAassistant

CLAassistant commented Jun 13, 2026 •

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@coderabbitai

coderabbitai Bot commented Jun 13, 2026 •

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

This pull request corrects a spelling inconsistency in variable field names across two Terraform modules. The variable attribute nat_rule_name_shortned is renamed to nat_rule_name_shortened in the dynamic_nat_rules variable schemas for both the Firewall-LB-DNAT-Rule-Collection and Firewall-Multiple-PIP modules. Resource configurations in both modules are updated to reference the corrected field name when constructing NAT rule names. The changes affect only variable definitions and their direct usages.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description check ✅ Passed The description comprehensively addresses all required template sections with specific details about the purpose, goals, approach, testing, and migration impact.
Linked Issues check ✅ Passed The PR directly resolves issue #157 by correcting the typo in 'nat_rule_name_shortned' to 'nat_rule_name_shortened' across all four affected files as requested.
Out of Scope Changes check ✅ Passed All code changes are directly related to the typo fix objective. The PR includes only the necessary variable definition updates and reference corrections, plus standard formatting via 'terraform fmt'.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check ✅ Passed The title accurately summarizes the main change: correcting a typo in the variable name from 'nat_rule_name_shortned' to 'nat_rule_name_shortened' across multiple files.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@kavix

kavix commented Jun 15, 2026

Copy link
Copy Markdown
Contributor Author

@Dilhasha The PR is now ready for re-review. Please take a look when you have a moment.
Thanks!

@athiththan11 athiththan11 changed the title Fix typo in variable name nat_rule_name_shortned to nat_rule_name_sho… Fix typo in variable name nat_rule_name_shortned Jun 17, 2026
@athiththan11
athiththan11 merged commit 74f4702 into wso2:main Jun 17, 2026
4 checks passed
@kavix
kavix deleted the feature/fix-nat-rule-name-typo branch June 17, 2026 05:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Typo in variable name in the module azurerm/Firewall-LB-DNAT-Rule-Collection

3 participants