Skip to content

Extract shared network configuration and environment policy helpers - #45180

Merged
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/444bf02d8543eaebdbd2ebc14b56d92ce6d2ad2f
Sep 13, 2026
Merged

copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/444bf02d8543eaebdbd2ebc14b56d92ce6d2ad2f

Conversation

@copyberry

@copyberry copyberry Bot commented Sep 13, 2026 •

Copy link
Copy Markdown

Extract shared network configuration and environment policy helpers

What changed

  • Introduce PreparedNetworkConfig to separate proxy preparation from applying managed network requirements, preserving preparation before permission fallback in local configuration loading.
  • Add helpers to build portable environment policies, retain selected or managed policies even when the proxy is disabled, and validate policies against the final permission profile using the execution resolver.
  • Strip listener addresses from environment network configuration and reject unsupported settings, malformed domain patterns, and invalid Unix socket paths.
  • Expose supporting managed-feature and permission-profile configuration helpers.

Testing

Add coverage for policy retention, listener removal, unsupported and malformed policies, and validation against permission profiles. Strengthen configuration tests to compare rebuilt proxy specifications and verify proxy configuration survives permission fallback.

…45180)

## What changed

- Introduce `PreparedNetworkConfig` to separate proxy preparation from applying managed network requirements, preserving preparation before permission fallback in local configuration loading.
- Add helpers to build portable environment policies, retain selected or managed policies even when the proxy is disabled, and validate policies against the final permission profile using the execution resolver.
- Strip listener addresses from environment network configuration and reject unsupported settings, malformed domain patterns, and invalid Unix socket paths.
- Expose supporting managed-feature and permission-profile configuration helpers.

## Testing

Add coverage for policy retention, listener removal, unsupported and malformed policies, and validation against permission profiles. Strengthen configuration tests to compare rebuilt proxy specifications and verify proxy configuration survives permission fallback.

GitOrigin-RevId: 444bf02d8543eaebdbd2ebc14b56d92ce6d2ad2f
@copyberry
copyberry Bot force-pushed the copyberry/codex-internal-to-codex-oss/444bf02d8543eaebdbd2ebc14b56d92ce6d2ad2f branch from 3577a65 to cfde11a Compare September 13, 2026 06:29
@copyberry
copyberry Bot merged commit cfde11a into main Sep 13, 2026
1 check failed
@copyberry
copyberry Bot deleted the copyberry/codex-internal-to-codex-oss/444bf02d8543eaebdbd2ebc14b56d92ce6d2ad2f branch September 13, 2026 06:30
@github-actions

Copy link
Copy Markdown
Contributor


Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.


I have read the CLA Document and I hereby sign the CLA


You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.

@github-actions github-actions Bot locked and limited conversation to collaborators Sep 13, 2026
@SDFJIJ
SDFJIJ deployed to issue-triage September 13, 2026 06:39 — with GitHub Actions Active
@SDFJIJ
SDFJIJ deployed to issue-triage September 13, 2026 06:39 — with GitHub Actions Active
@SDFJIJ
SDFJIJ deployed to issue-triage September 13, 2026 06:39 — with GitHub Actions Active
@SDFJIJ
SDFJIJ deployed to issue-triage September 13, 2026 06:40 — with GitHub Actions Active

This branch was successfully deployed

1 active deployment
issue-triage — cfde11a2 Deployed Sep 13, 2026 by SDFJIJ via Identify potential duplicates (open issues fallback) #45130
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.

2 participants