Skip to content

Handle exec-server network policy requests in the client#35359

Merged
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/6ee2a0139e0f8170cf1a5cdd646bd811b7d3c1f6
Jul 25, 2026
Merged

Handle exec-server network policy requests in the client#35359
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/6ee2a0139e0f8170cf1a5cdd646bd811b7d3c1f6

Conversation

@copyberry

@copyberry copyberry Bot commented Jul 25, 2026

Copy link
Copy Markdown

Handle exec-server network policy requests in the client

What changed

  • Add client-side handling for exec-server network policy requests, including request validation, per-process decision routing, and allow, deny, or ask responses.
  • Bound concurrent callbacks and fail closed when requests are invalid, exceed capacity, time out, or outlive their process session.
  • Replace the callback opt-in flag with a launch-level policyDecisionTimeoutMs value and include transport overhead in the executor timeout.
  • Preserve callback admission limits across connection recovery and clean up policy state when process startup is abandoned or a session ends.

Testing

  • Cover decision forwarding, invalid and excess requests, cancellation during process cleanup, abandoned starts, timeout propagation, and launch-config serialization.

## What changed

- Add client-side handling for exec-server network policy requests, including request validation, per-process decision routing, and allow, deny, or ask responses.
- Bound concurrent callbacks and fail closed when requests are invalid, exceed capacity, time out, or outlive their process session.
- Replace the callback opt-in flag with a launch-level `policyDecisionTimeoutMs` value and include transport overhead in the executor timeout.
- Preserve callback admission limits across connection recovery and clean up policy state when process startup is abandoned or a session ends.

## Testing

- Cover decision forwarding, invalid and excess requests, cancellation during process cleanup, abandoned starts, timeout propagation, and launch-config serialization.

GitOrigin-RevId: 6ee2a0139e0f8170cf1a5cdd646bd811b7d3c1f6
@copyberry
copyberry Bot force-pushed the copyberry/codex-internal-to-codex-oss/6ee2a0139e0f8170cf1a5cdd646bd811b7d3c1f6 branch from e13f530 to 3a08af4 Compare July 25, 2026 16:03
@copyberry
copyberry Bot merged commit 3a08af4 into main Jul 25, 2026
14 of 32 checks passed
@copyberry
copyberry Bot deleted the copyberry/codex-internal-to-codex-oss/6ee2a0139e0f8170cf1a5cdd646bd811b7d3c1f6 branch July 25, 2026 16:04
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 25, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants