Skip to content

feat(network): add organization outbound routing - #7786

Open
waleedlatif1 wants to merge 1 commit into
codex/http-transport-foundationfrom
codex/organization-egress
Open

feat(network): add organization outbound routing#7786
waleedlatif1 wants to merge 1 commit into
codex/http-transport-foundationfrom
codex/organization-egress

Conversation

@waleedlatif1

@waleedlatif1 waleedlatif1 commented Sep 11, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Add organization outbound routing to the existing validated HTTP helpers and guarded SDK transports, with isolated pools, TLS CONNECT, canonical ownership and fail-closed routing.
  • Migrate managed requests to explicit destination profiles and extend the existing boundary audit. Preserve supported streaming, uploads, retries, model endpoints and deployment proxies.
  • Combine session policies and outbound IP addresses in Security settings using shared components. Keep SSO separate and document transport coverage.
  • Keep credentials and organization assignments in operator configuration. Assigned routes reject unsupported remote sandbox, raw database and SDK transports; initial account OAuth and SDK-owned control traffic remain outside coverage. Activation is separate.
  • Depends on improvement(network): consolidate validated HTTP transports #7795. This PR contains the organization feature; the prerequisite contains reusable HTTP transport changes.

Type of Change

  • New feature

Testing

  • Full combined application regression passed 50,986 tests with 185 skipped before the staging sync. After the sync and final CONNECT-port fix, 2,223 focused tests passed with 18 skipped.
  • All 26 workspace type checks, lint, 46 repository audits, block registry and docs manifest checks passed. Real Node and Bun socket tests cover TLS identity, destination pinning, default ports, proxies and cancellation.
  • Security settings were verified locally with synthetic data. CI on the new commit, deployed app/worker acceptance and connection coverage remain release gates.

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

@waleedlatif1
waleedlatif1 requested a review from a team as a code owner September 11, 2026 21:01
@vercel

vercel Bot commented Sep 11, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
docs Ready Ready Preview Sep 12, 2026 9:59am UTC

Request Review

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile-apps please review this PR.

@greptile-apps

greptile-apps Bot commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 5/5

The PR appears safe to merge; no new blocking or independently actionable defect remains in the reviewed changes.

Summary

  • Migrates connectors, tools, providers, webhooks, OAuth operations, and background jobs onto explicit destination profiles and shared outbound scope boundaries.
  • Adds an administrator-only Security settings surface for session policies and configured outbound IPv4 addresses.
  • Extends deployment documentation, transport audits, routing tests, and integration regression coverage.
  • Adds matching database schema and migration support for binary embedding candidate indexes.

Diagram

%%{init: {'theme': 'neutral'}}%%
flowchart LR
  Resource[Canonical workspace or organization] --> Scope[Outbound organization scope]
  Scope --> Route{Resolve route}
  Route -->|Direct| EnvProxy{Environment proxy configured?}
  EnvProxy -->|No| Destination[Validated destination]
  EnvProxy -->|Yes| Proxy[Validated proxy CONNECT]
  Route -->|Gateway| Gateway[Organization gateway CONNECT]
  Route -->|Blocked or missing| Reject[Fail closed]
  Proxy --> TLS[TLS verifies original hostname]
  Gateway --> TLS
  TLS --> Destination
Loading

Reviews (8) · Last reviewed commit: "feat(network): add organization outbound..."

Comment thread apps/sim/lib/credential-groups/standard-oauth-provider.ts Outdated
Comment thread apps/sim/lib/workflows/executor/execution-core.ts
@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile-apps please review this PR.

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

Comment thread apps/sim/lib/core/config/appconfig.ts Outdated
@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile-apps please review this PR.

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile-apps please review this PR.

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile-apps please review this PR.

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Sep 12, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1
waleedlatif1 force-pushed the codex/organization-egress branch from 94d423e to 3b319c5 Compare September 12, 2026 09:57
@waleedlatif1
waleedlatif1 changed the base branch from staging to codex/http-transport-foundation September 12, 2026 09:57
@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Sep 12, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

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.

1 participant