Skip to content

chore: egress allowlist openapi changes - #163

Open
sk-portkey wants to merge 1 commit into
masterfrom
chore/access-policies
Open

sk-portkey wants to merge 1 commit into
masterfrom
chore/access-policies

Conversation

@sk-portkey

Copy link
Copy Markdown
Collaborator

No description provided.

Copilot AI balanced review requested due to automatic review settings October 9, 2026 14:42

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟡 Changes recommended

Navigation registration is missing and the update schema permits invalid empty requests.

2 open findings
What changed in this PR

Adds OpenAPI support for managing organization egress allowlist entries.

Changes:

  • Defines CRUD and listing endpoints.
  • Adds request, domain, and response schemas.
  • Provides hosted and self-hosted cURL examples.
File Description
openapi.yaml Adds egress allowlist API definitions and schemas.

🧠 Review effort: Balanced


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread openapi.yaml
Comment thread openapi.yaml
naresh4dev
naresh4dev previously approved these changes Oct 9, 2026
Copilot AI balanced review requested due to automatic review settings October 10, 2026 11:16
@sk-portkey
sk-portkey force-pushed the chore/access-policies branch from 14b5362 to 0f92227 Compare October 10, 2026 11:16
@sk-portkey sk-portkey changed the title chore: access policies openapi changes chore: egress allowlist openapi changes Oct 10, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟢 Approval recommended

The new API definitions are valid, internally consistent, and follow existing specification patterns.

0 open findings

2 resolved since last review

🧠 Review effort: Balanced

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