Skip to content

Conversation

@zry98
Copy link

@zry98 zry98 commented Dec 22, 2025

  • I understand that this repository is auto-generated and my pull request may not be merged

Changes being requested

#6570

Fix resource cloudflare_list CIDR validation errors like:

│ Error: IP address must be normalized
│
│   with cloudflare_list.ip_whitelist,
│   on lists.tf line 8, in resource "cloudflare_list" "ip_whitelist":
│    8:   items = [
│    9:     {
│   10:       ip = "1.1.1.0/24"
│   11:       comment = "Test IP addr"
│   12:     },
│   13:     {
│   14:       ip = "2400:cb00::/32"
│   15:       comment = "Test IP CIDR"
│   16:     },
│   17:   ]
│
│ IP address "1.1.1.0/24" must be normalized: "1.1.1.0"

│ Error: IP address must be normalized
│
│   with cloudflare_list.ip_whitelist,
│   on lists.tf line 8, in resource "cloudflare_list" "ip_whitelist":
│    8:   items = [
│    9:     {
│   10:       ip = "1.1.1.0/24"
│   11:       comment = "Test IP addr"
│   12:     },
│   13:     {
│   14:       ip = "2400:cb00::/32"
│   15:       comment = "Test IP CIDR"
│   16:     },
│   17:   ]
│
│ IP address "2400:cb00::/32" must be normalized: "2400:cb00::"

Due to the normalization validation treating CIDR as IP address.

Acceptance test run results

  • I have added or updated acceptance tests for my changes
  • I have run acceptance tests for my changes and included the results below

Steps to run acceptance tests

Test output

Additional context & links

@dirnikk
Copy link

dirnikk commented Dec 22, 2025

The latest version 5.15.0 experience the failure, however with version 5.14.0 I am able to deploy CIDRs in list object without any issues.

@zry98
Copy link
Author

zry98 commented Dec 22, 2025

The latest version 5.15.0 experience the failure, however with version 5.14.0 I am able to deploy CIDRs in list object without any issues.

@dirnikk Exactly, it was introduced in the commit c2dcd1a

Copy link

@dirnikk dirnikk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This PR is a must.

@mgirouard
Copy link
Contributor

cc @psutherland-cloudflare

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.

3 participants