Skip to content

feat(github_ip_ranges): support for Web and API IP Ranges#1220

Merged
kfcampbell merged 4 commits intointegrations:mainfrom
Jamian:ipranges-meta-api-web-blocks
Aug 30, 2022
Merged

feat(github_ip_ranges): support for Web and API IP Ranges#1220
kfcampbell merged 4 commits intointegrations:mainfrom
Jamian:ipranges-meta-api-web-blocks

Conversation

@Jamian
Copy link
Copy Markdown
Contributor

@Jamian Jamian commented Jul 15, 2022

Adding the following attributes to the github_ip_ranges data resource:

  • web
  • web_ipv4
  • web_ipv6
  • api
  • api_ipv4
  • api_ipv6

Somebody else had an old PR for this here - #483 which was blocked pending GraphQL API support.

@kfcampbell
Copy link
Copy Markdown
Contributor

Hmm, looks like Web doesn't exist on the github.com/google/go-github@v45.2.0 type APIMeta.

Copy link
Copy Markdown
Contributor

@kfcampbell kfcampbell left a comment

Choose a reason for hiding this comment

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

I've locally confirmed acceptance testing is passing. Thank you for the contributions! This functionality will be released this week.

@kfcampbell kfcampbell merged commit c753688 into integrations:main Aug 30, 2022
kazaker pushed a commit to auto1-oss/terraform-provider-github that referenced this pull request Dec 28, 2022
…ns#1220)

* feat(github_ip_ranges): support for web and api CIDR blocks from meta API.

* fix: Api --> API as per sdk changes. Re-adding accidentally removed Pages.

Co-authored-by: Keegan Campbell <me@kfcampbell.com>
avidspartan1 pushed a commit to avidspartan1/terraform-provider-github that referenced this pull request Feb 5, 2024
…ns#1220)

* feat(github_ip_ranges): support for web and api CIDR blocks from meta API.

* fix: Api --> API as per sdk changes. Re-adding accidentally removed Pages.

Co-authored-by: Keegan Campbell <me@kfcampbell.com>
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.

2 participants