Skip to content

Commit

Permalink
Add support for packages IP ranges in github_ip_ranges (#1958)
Browse files Browse the repository at this point in the history
* Add support for packages IP ranges in github_ip_ranges

* Update docks for packages IP ranges in github_ip_ranges

* gofmt

* Add tests for packages IP ranges in github_ip_ranges

* Update tests for packages IP ranges in github_ip_ranges

---------

Co-authored-by: Keegan Campbell <me@kfcampbell.com>
  • Loading branch information
Nmishin and kfcampbell committed Feb 2, 2024
1 parent ddb0c3b commit 970dd20
Show file tree
Hide file tree
Showing 3 changed files with 31 additions and 0 deletions.
25 changes: 25 additions & 0 deletions github/data_source_github_ip_ranges.go
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,11 @@ func dataSourceGithubIpRanges() *schema.Resource {
Computed: true,
Elem: &schema.Schema{Type: schema.TypeString},
},
"packages": {
Type: schema.TypeList,
Computed: true,
Elem: &schema.Schema{Type: schema.TypeString},
},
"pages": {
Type: schema.TypeList,
Computed: true,
Expand Down Expand Up @@ -72,6 +77,11 @@ func dataSourceGithubIpRanges() *schema.Resource {
Computed: true,
Elem: &schema.Schema{Type: schema.TypeString},
},
"packages_ipv4": {
Type: schema.TypeList,
Computed: true,
Elem: &schema.Schema{Type: schema.TypeString},
},
"pages_ipv4": {
Type: schema.TypeList,
Computed: true,
Expand Down Expand Up @@ -112,6 +122,11 @@ func dataSourceGithubIpRanges() *schema.Resource {
Computed: true,
Elem: &schema.Schema{Type: schema.TypeString},
},
"packages_ipv6": {
Type: schema.TypeList,
Computed: true,
Elem: &schema.Schema{Type: schema.TypeString},
},
"pages_ipv6": {
Type: schema.TypeList,
Computed: true,
Expand Down Expand Up @@ -154,6 +169,11 @@ func dataSourceGithubIpRangesRead(d *schema.ResourceData, meta interface{}) erro
return err
}

cidrPackagesIpv4, cidrPackagesIpv6, err := splitIpv4Ipv6Cidrs(&api.Packages)
if err != nil {
return err
}

cidrPagesIpv4, cidrPagesIpv6, err := splitIpv4Ipv6Cidrs(&api.Pages)
if err != nil {
return err
Expand Down Expand Up @@ -197,6 +217,11 @@ func dataSourceGithubIpRangesRead(d *schema.ResourceData, meta interface{}) erro
d.Set("git_ipv4", cidrGitIpv4)
d.Set("git_ipv6", cidrGitIpv6)
}
if len(api.Packages) > 0 {
d.Set("packages", api.Packages)
d.Set("packages_ipv4", cidrPackagesIpv4)
d.Set("packages_ipv6", cidrPackagesIpv6)
}
if len(api.Pages) > 0 {
d.Set("pages", api.Pages)
d.Set("pages_ipv4", cidrPagesIpv4)
Expand Down
3 changes: 3 additions & 0 deletions github/data_source_github_ip_ranges_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@ func TestAccGithubIpRangesDataSource(t *testing.T) {
resource.TestCheckResourceAttrSet("data.github_ip_ranges.test", "git.#"),
resource.TestCheckResourceAttrSet("data.github_ip_ranges.test", "api.#"),
resource.TestCheckResourceAttrSet("data.github_ip_ranges.test", "web.#"),
resource.TestCheckResourceAttrSet("data.github_ip_ranges.test", "packages.#"),
resource.TestCheckResourceAttrSet("data.github_ip_ranges.test", "pages.#"),
resource.TestCheckResourceAttrSet("data.github_ip_ranges.test", "importer.#"),
resource.TestCheckResourceAttrSet("data.github_ip_ranges.test", "actions.#"),
Expand All @@ -25,6 +26,7 @@ func TestAccGithubIpRangesDataSource(t *testing.T) {
resource.TestCheckResourceAttrSet("data.github_ip_ranges.test", "git_ipv4.#"),
resource.TestCheckResourceAttrSet("data.github_ip_ranges.test", "api_ipv4.#"),
resource.TestCheckResourceAttrSet("data.github_ip_ranges.test", "web_ipv4.#"),
resource.TestCheckResourceAttrSet("data.github_ip_ranges.test", "packages_ipv4.#"),
resource.TestCheckResourceAttrSet("data.github_ip_ranges.test", "pages_ipv4.#"),
resource.TestCheckResourceAttrSet("data.github_ip_ranges.test", "importer_ipv4.#"),
resource.TestCheckResourceAttrSet("data.github_ip_ranges.test", "actions_ipv4.#"),
Expand All @@ -33,6 +35,7 @@ func TestAccGithubIpRangesDataSource(t *testing.T) {
resource.TestCheckResourceAttrSet("data.github_ip_ranges.test", "git_ipv6.#"),
resource.TestCheckResourceAttrSet("data.github_ip_ranges.test", "api_ipv6.#"),
resource.TestCheckResourceAttrSet("data.github_ip_ranges.test", "web_ipv6.#"),
resource.TestCheckResourceAttrSet("data.github_ip_ranges.test", "packages_ipv6.#"),
resource.TestCheckResourceAttrSet("data.github_ip_ranges.test", "pages_ipv6.#"),
resource.TestCheckResourceAttrSet("data.github_ip_ranges.test", "importer_ipv6.#"),
resource.TestCheckResourceAttrSet("data.github_ip_ranges.test", "actions_ipv6.#"),
Expand Down
3 changes: 3 additions & 0 deletions website/docs/d/ip_ranges.html.markdown
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,9 @@ data "github_ip_ranges" "test" {}
* `api` - An Array of IP addresses in CIDR format for the GitHub API.
* `api_ipv4` - A subset of the `api` array that contains IP addresses in IPv4 CIDR format.
* `api_ipv6` - A subset of the `api` array that contains IP addresses in IPv6 CIDR format.
* `packages` - An Array of IP addresses in CIDR format specifying the A records for GitHub Packages.
* `packages_ipv4` - A subset of the `packages` array that contains IP addresses in IPv4 CIDR format.
* `packages_ipv6` - A subset of the `packages` array that contains IP addresses in IPv6 CIDR format.
* `pages` - An Array of IP addresses in CIDR format specifying the A records for GitHub Pages.
* `pages_ipv4` - A subset of the `pages` array that contains IP addresses in IPv4 CIDR format.
* `pages_ipv6` - A subset of the `pages` array that contains IP addresses in IPv6 CIDR format.
Expand Down

0 comments on commit 970dd20

Please sign in to comment.