Skip to content

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

Open
waleedlatif1 wants to merge 9 commits into
stagingfrom
codex/organization-egress
Open

feat(network): add organization outbound routing#7786
waleedlatif1 wants to merge 9 commits into
stagingfrom
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.
  • Remove the additional fetch entry point and migrate callers to explicit destination profiles. Preserve streaming, multipart uploads, retry budgets, configured private model endpoints and supported deployment proxies. Extend the existing audit to reject native fetch aliases and SDK handoffs.
  • Combine session policies and outbound IP addresses in Security settings using shared components. Preserve the existing session URL and feature gates, keep SSO separate, and document connection coverage.
  • Keep gateway credentials and organization assignments in operator configuration. Configured routes reject unsupported transports, including remote sandbox execution and raw database connections; initial account OAuth and SDK-owned control traffic remain outside coverage. Production activation is separate.

Type of Change

  • New feature

Testing

  • 20,889 regression tests passed; 98 skipped. Includes real Node and Bun gateway/proxy connections, TLS identity, destination pinning, organization isolation, redirects, streaming, decompression limits, cancellation and self-hosted model endpoints.
  • All 26 workspace type checks, lint, 46 repository audits, block registry and docs manifest checks passed. Nine boundary-audit tests passed.
  • Security settings were verified locally with synthetic data. Production app/worker acceptance and connection coverage remain release gates; local tests do not establish deployed routing.

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 12:08am 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 actionable new issue or outstanding previous finding remains.

Summary

  • Routes supported outbound requests through organization-scoped gateway transports with fail-closed enforcement.
  • Propagates canonical organization ownership through authorized operations, background work, OAuth enrollment, and custom-block execution.
  • Adds bounded configuration snapshot caching and isolates gateway connection pools.
  • Combines session policies and read-only outbound IP information under Security settings while retaining legacy URL aliases.
  • Documents feature availability, routing limitations, and outbound allowlisting guidance.

Diagram

%%{init: {'theme': 'neutral'}}%%
flowchart LR
  Operation[Authorized operation or background job] --> Scope[Canonical organization scope]
  Scope --> Policy[Resolve outbound routing policy]
  Policy -->|Direct or unconfigured| Direct[Existing network path]
  Policy -->|Gateway required| Gateway[Organization-isolated gateway transport]
  Policy -->|Blocked or unsupported| Reject[Fail closed]
  Gateway --> External[External HTTP service]
  Admin[Organization administrator] --> Security[Security settings]
  Security --> Status[Routing state and outbound IPs]
Loading

Reviews (7) · Last reviewed commit: "improvement(settings): document organiza..."

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

@greptile-apps

greptile-apps Bot commented Sep 12, 2026

Copy link
Copy Markdown
Contributor

Too many files changed for review (529 files, 500 file limit).

@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

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