Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add default-domains-netblock for google_netblock_ip_ranges data source #18290

Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions go.mod
Original file line number Diff line number Diff line change
Expand Up @@ -97,6 +97,7 @@ require (
go.opentelemetry.io/otel v1.24.0 // indirect
go.opentelemetry.io/otel/metric v1.24.0 // indirect
go.opentelemetry.io/otel/trace v1.24.0 // indirect
go4.org/netipx v0.0.0-20231129151722-fdeea329fbba // indirect
golang.org/x/crypto v0.22.0 // indirect
golang.org/x/mod v0.17.0 // indirect
golang.org/x/sync v0.7.0 // indirect
Expand Down
2 changes: 2 additions & 0 deletions go.sum
Original file line number Diff line number Diff line change
Expand Up @@ -287,6 +287,8 @@ go.opentelemetry.io/otel/trace v1.24.0/go.mod h1:HPc3Xr/cOApsBI154IU0OI0HJexz+aw
go.uber.org/atomic v1.4.0/go.mod h1:gD2HeocX3+yG+ygLZcrzQJaqmWj9AIm7n08wl/qW/PE=
go.uber.org/multierr v1.1.0/go.mod h1:wR5kodmAFQ0UK8QlbwjlSNy0Z68gJhDJUG5sjR94q/0=
go.uber.org/zap v1.10.0/go.mod h1:vwi/ZaCAaUcBkycHslxD9B2zi4UTXhF60s6SWpuDF0Q=
go4.org/netipx v0.0.0-20231129151722-fdeea329fbba h1:0b9z3AuHCjxk0x/opv64kcgZLBseWJUpBw5I82+2U4M=
go4.org/netipx v0.0.0-20231129151722-fdeea329fbba/go.mod h1:PLyyIXexvUFg3Owu6p/WfdlivPbZJsZdgWZlrGope/Y=
golang.org/x/crypto v0.0.0-20190308221718-c2843e01d9a2/go.mod h1:djNgcEr1/C05ACkg1iLfiJU5Ep61QUkGW8qpdssI0+w=
golang.org/x/crypto v0.0.0-20191011191535-87dc89f01550/go.mod h1:yigFU9vqHzYiE8UmvKecakEJjdnWj3jj499lnFckfCI=
golang.org/x/crypto v0.0.0-20200622213623-75b288015ac9/go.mod h1:LzIPMQfyMNhhGPhUkYOs5KpL4U8rLKemX1yGLhDgUto=
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,8 +7,11 @@ import (
"fmt"
"io/ioutil"
"net/http"
"net/netip"
"sort"

"github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema"
"go4.org/netipx"
)

type googRanges struct {
Expand All @@ -22,6 +25,11 @@ type prefixes struct {
Ipv6Prefix string `json:"ipv6Prefix"`
}

const (
CLOUD_NETBLOCK_URL = "https://www.gstatic.com/ipranges/cloud.json"
GOOGLE_NETBLOCK_URL = "https://www.gstatic.com/ipranges/goog.json"
)

func DataSourceGoogleNetblockIpRanges() *schema.Resource {
return &schema.Resource{
Read: dataSourceGoogleNetblockIpRangesRead,
Expand Down Expand Up @@ -60,7 +68,6 @@ func dataSourceGoogleNetblockIpRangesRead(d *schema.ResourceData, meta interface
// Dynamic ranges
case "cloud-netblocks":
// https://cloud.google.com/compute/docs/faq#find_ip_range
const CLOUD_NETBLOCK_URL = "https://www.gstatic.com/ipranges/cloud.json"
CidrBlocks, err := getCidrBlocksFromUrl(CLOUD_NETBLOCK_URL)

if err != nil {
Expand All @@ -77,12 +84,35 @@ func dataSourceGoogleNetblockIpRangesRead(d *schema.ResourceData, meta interface
}
case "google-netblocks":
// https://cloud.google.com/vpc/docs/configure-private-google-access?hl=en#ip-addr-defaults
const GOOGLE_NETBLOCK_URL = "https://www.gstatic.com/ipranges/goog.json"
CidrBlocks, err := getCidrBlocksFromUrl(GOOGLE_NETBLOCK_URL)

if err != nil {
return err
}
if err := d.Set("cidr_blocks", CidrBlocks["cidr_blocks"]); err != nil {
return fmt.Errorf("Error setting cidr_blocks: %s", err)
}
if err := d.Set("cidr_blocks_ipv4", CidrBlocks["cidr_blocks_ipv4"]); err != nil {
return fmt.Errorf("Error setting cidr_blocks_ipv4: %s", err)
}
if err := d.Set("cidr_blocks_ipv6", CidrBlocks["cidr_blocks_ipv6"]); err != nil {
return fmt.Errorf("Error setting cidr_blocks_ipv6: %s", err)
}
case "default-domains-netblocks":
// https: //cloud.google.com/vpc/docs/configure-private-google-access#ip-addr-defaults
googleBlocks, err := getCidrBlocksFromUrl(GOOGLE_NETBLOCK_URL)
if err != nil {
return err
}
cloudBlocks, err := getCidrBlocksFromUrl(CLOUD_NETBLOCK_URL)
if err != nil {
return err
}
CidrBlocks, err := getCidrsDifference(googleBlocks, cloudBlocks)
if err != nil {
return err
}

if err := d.Set("cidr_blocks", CidrBlocks["cidr_blocks"]); err != nil {
return fmt.Errorf("Error setting cidr_blocks: %s", err)
}
Expand Down Expand Up @@ -201,3 +231,42 @@ func getCidrBlocksFromUrl(url string) (map[string][]string, error) {

return cidrBlocks, nil
}

func getCidrsDifference(reference, excluded map[string][]string) (map[string][]string, error) {
result := make(map[string][]string)

for blockName := range reference {
var ipSetBuilder netipx.IPSetBuilder
for _, cidr := range reference[blockName] {
net, err := netip.ParsePrefix(cidr)
if err != nil {
return result, err
}
ipSetBuilder.AddPrefix(net)
}

for _, cidr := range excluded[blockName] {
net, err := netip.ParsePrefix(cidr)
if err != nil {
return result, err
}
ipSetBuilder.RemovePrefix(net)
}

ipSet, err := ipSetBuilder.IPSet()
if err != nil {
return result, err
}

var ipRangeStrings []string
for _, ipRange := range ipSet.Prefixes() {
ipRangeStrings = append(ipRangeStrings, ipRange.String())
}

sort.Strings(ipRangeStrings)

result[blockName] = ipRangeStrings
}

return result, nil
}
Original file line number Diff line number Diff line change
Expand Up @@ -51,6 +51,24 @@ func TestAccDataSourceGoogleNetblockIpRanges_basic(t *testing.T) {
"cidr_blocks_ipv6.0", regexp.MustCompile("^(?:[0-9a-fA-F]{1,4}:){1,2}.*/[0-9]{1,3}$")),
),
},
{
Config: testAccNetblockIpRangesConfig_defaultdomains,
Check: resource.ComposeTestCheckFunc(
// Default domains netblocks
resource.TestMatchResourceAttr("data.google_netblock_ip_ranges.defaultdomains",
"cidr_blocks.#", regexp.MustCompile(("^[1-9]+[0-9]*$"))),
resource.TestMatchResourceAttr("data.google_netblock_ip_ranges.defaultdomains",
"cidr_blocks.0", regexp.MustCompile("^(?:[0-9a-fA-F./:]{1,4}){1,2}.*/[0-9]{1,3}$")),
resource.TestMatchResourceAttr("data.google_netblock_ip_ranges.defaultdomains",
"cidr_blocks_ipv4.#", regexp.MustCompile(("^[1-9]+[0-9]*$"))),
resource.TestMatchResourceAttr("data.google_netblock_ip_ranges.defaultdomains",
"cidr_blocks_ipv4.0", regexp.MustCompile("^(?:[0-9]{1,3}.){3}[0-9]{1,3}/[0-9]{1,2}$")),
resource.TestMatchResourceAttr("data.google_netblock_ip_ranges.defaultdomains",
"cidr_blocks_ipv6.#", regexp.MustCompile(("^[1-9]+[0-9]*$"))),
resource.TestMatchResourceAttr("data.google_netblock_ip_ranges.defaultdomains",
"cidr_blocks_ipv6.0", regexp.MustCompile("^(?:[0-9a-fA-F]{1,4}:){1,2}.*/[0-9]{1,3}$")),
),
},
{
Config: testAccNetblockIpRangesConfig_restricted,
Check: resource.ComposeTestCheckFunc(
Expand Down Expand Up @@ -143,6 +161,12 @@ data "google_netblock_ip_ranges" "google" {
}
`

const testAccNetblockIpRangesConfig_defaultdomains = `
data "google_netblock_ip_ranges" "defaultdomains" {
range_type = "default-domains-netblocks"
}
`

const testAccNetblockIpRangesConfig_restricted = `
data "google_netblock_ip_ranges" "restricted" {
range_type = "restricted-googleapis"
Expand Down