Skip to content

feat: add checks for ip whitelist#9

Open
gusfcarvalho wants to merge 2 commits intomainfrom
gc-feat-more-checks
Open

feat: add checks for ip whitelist#9
gusfcarvalho wants to merge 2 commits intomainfrom
gc-feat-more-checks

Conversation

@gusfcarvalho
Copy link
Copy Markdown
Contributor

@gusfcarvalho gusfcarvalho commented May 5, 2026

feat: add checks for SSO

feat: add checks for organization IP allow-list collection

feat: add checks for sso

fix: use new configurations endpoint for dependabot and secret scanning
Signed-off-by: Gustavo Carvalho <gustavo.carvalho@container-solutions.com>
Copilot AI review requested due to automatic review settings May 5, 2026 12:51
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR expands the GitHub organization data collected by the plugin to support policies that validate SSO enforcement and organization IP allow-list (whitelist) configuration.

Changes:

  • Extend the collected dataset to include org admin members, SSO configuration, and IP allow-list entries.
  • Add REST-based SSO config retrieval and GraphQL-based IP allow-list retrieval.
  • Add additional “collect data” step descriptions for the newly fetched artifacts.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread internal/data.go Outdated
Comment thread internal/data.go Outdated
Comment thread internal/data.go Outdated
Comment thread internal/data.go Outdated
Comment thread internal/data.go Outdated
Comment thread internal/data.go Outdated
Comment thread internal/data.go Outdated
Comment thread internal/data.go Outdated
Comment thread internal/data.go
Signed-off-by: Gustavo Carvalho <gustavo.carvalho@container-solutions.com>
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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