Skip to content

Allow chatgpt.com in Codex smoke workflow AWF domain allowlist#2102

Open
Copilot wants to merge 2 commits intomainfrom
copilot/awf-fix-chatgpt-firewall-issue
Open

Allow chatgpt.com in Codex smoke workflow AWF domain allowlist#2102
Copilot wants to merge 2 commits intomainfrom
copilot/awf-fix-chatgpt-firewall-issue

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 19, 2026

Codex (gpt-5.3-codex) runs were intermittently degraded by AWF denying CONNECT chatgpt.com:443 (TCP_DENIED), while primary api.openai.com auth failures remained a separate credential concern. This PR closes the firewall allowlist gap for the Codex smoke path.

  • Workflow source update (smoke-codex.md)

    • Added chatgpt.com to network.allowed for the Codex smoke workflow.
  • Compiled workflow sync (smoke-codex.lock.yml)

    • Propagated the same domain into compiled/runtime allowlists so execution and metadata are aligned:
      • GH_AW_INFO_ALLOWED_DOMAINS
      • AWF CLI --allow-domains list
      • GH_AW_ALLOWED_DOMAINS env values used in workflow steps
  • Behavioral impact

    • Codex startup/plugin-cache traffic to chatgpt.com is now permitted within this workflow’s AWF policy, reducing non-auth-related connection denials.
# .github/workflows/smoke-codex.md
network:
  allowed:
    - defaults
    - chatgpt.com
    - github
    - playwright

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • https://api.github.com/repos/actions/github-script/git/ref/tags/v9
    • Triggering command: /usr/bin/gh gh api /repos/actions/github-script/git/ref/tags/v9 --jq .object.sha (http block)
  • https://api.github.com/repos/github/gh-aw-actions/git/ref/tags/v0.68.3
    • Triggering command: /usr/bin/gh gh api /repos/github/gh-aw-actions/git/ref/tags/v0.68.3 --jq .object.sha (http block)

If you need me to access, download, or install something from one of these locations, you can either:

Copilot AI changed the title [WIP] Fix firewall blocking chatgpt.com for codex plugin caching Allow chatgpt.com in Codex smoke workflow AWF domain allowlist Apr 19, 2026
Copilot AI requested a review from lpcox April 19, 2026 21:54
@lpcox lpcox marked this pull request as ready for review April 19, 2026 22:07
@lpcox lpcox requested a review from Mossaka as a code owner April 19, 2026 22:07
Copilot AI review requested due to automatic review settings April 19, 2026 22:07
@github-actions
Copy link
Copy Markdown
Contributor

✅ Coverage Check Passed

Overall Coverage

Metric Base PR Delta
Lines 84.31% 84.39% 📈 +0.08%
Statements 83.55% 83.63% 📈 +0.08%
Functions 87.46% 87.46% ➡️ +0.00%
Branches 74.72% 74.76% 📈 +0.04%
📁 Per-file Coverage Changes (1 files)
File Lines (Before → After) Statements (Before → After)
src/docker-manager.ts 86.8% → 87.1% (+0.30%) 86.4% → 86.7% (+0.29%)

Coverage comparison generated by scripts/ci/compare-coverage.ts

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the Codex smoke agentic workflow allowlist to permit outbound connectivity to chatgpt.com, addressing intermittent CONNECT chatgpt.com:443 firewall denials during Codex runs.

Changes:

  • Added chatgpt.com to .github/workflows/smoke-codex.md under network.allowed.
  • Synced the compiled workflow .github/workflows/smoke-codex.lock.yml to include chatgpt.com in GH_AW_INFO_ALLOWED_DOMAINS, the AWF CLI --allow-domains list, and GH_AW_ALLOWED_DOMAINS env values.
  • Minor markdown formatting fix in the workflow task instructions list.
Show a summary per file
File Description
.github/workflows/smoke-codex.md Adds chatgpt.com to the workflow’s network allowlist (source) and fixes a list indentation.
.github/workflows/smoke-codex.lock.yml Propagates chatgpt.com into the compiled/runtime allowlist variables and AWF invocation.

Copilot's findings

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

  • Files reviewed: 2/2 changed files
  • Comments generated: 0

@github-actions
Copy link
Copy Markdown
Contributor

Smoke Test Results: PASS

  • GitHub MCP: [docs] docs: update --image-tag CLI reference for digest-aware format ✅
  • GitHub MCP: Add digest-aware AWF runtime image pinning via image-tag metadata ✅
  • Playwright: GitHub page title contains "GitHub" ✅
  • File Writing: Test file created successfully ✅

💥 [THE END] — Illustrated by Smoke Claude

@github-actions

This comment has been minimized.

@github-actions github-actions bot mentioned this pull request Apr 19, 2026
@github-actions
Copy link
Copy Markdown
Contributor

Smoke Test: GitHub Actions Services Connectivity

Check Result
Redis PING (host.docker.internal:6379) redis-cli not available (cannot install — no apt access in sandbox)
pg_isready (host.docker.internal:5432) accepting connections
psql SELECT 1 (smoketest db, user postgres) ✅ Returns 1

2/3 checks passed. Redis check could not be performed due to missing redis-cli tool in the sandbox environment.

🔌 Service connectivity validated by Smoke Services

@github-actions
Copy link
Copy Markdown
Contributor

🏗️ Build Test Suite Results

Ecosystem Project Build/Install Tests Status
Bun elysia 1/1 passed ✅ PASS
Bun hono 1/1 passed ✅ PASS
C++ fmt N/A ✅ PASS
C++ json N/A ✅ PASS
Deno oak N/A 1/1 passed ✅ PASS
Deno std N/A 1/1 passed ✅ PASS
.NET hello-world N/A ✅ PASS
.NET json-parse N/A ✅ PASS
Go color 1/1 passed ✅ PASS
Go env 1/1 passed ✅ PASS
Go uuid 1/1 passed ✅ PASS
Java gson 1/1 passed ✅ PASS
Java caffeine 1/1 passed ✅ PASS
Node.js clsx passed ✅ PASS
Node.js execa passed ✅ PASS
Node.js p-limit passed ✅ PASS
Rust fd 1/1 passed ✅ PASS
Rust zoxide 1/1 passed ✅ PASS

Overall: 8/8 ecosystems passed — ✅ PASS

Generated by Build Test Suite for issue #2102 · ● 422.6K ·

@github-actions
Copy link
Copy Markdown
Contributor

🤖 Smoke Test Results

Test Status
MCP: Latest merged PR — "[docs] docs: update --image-tag CLI reference for digest-aware format" (#2095)
GitHub.com connectivity (HTTP 200)
File write/read (smoke-test-copilot-24640001488.txt)

Overall: PASS

PR author: @app/copilot-swe-agent — Assignees: @lpcox, @Copilot

📰 BREAKING: Report filed by Smoke Copilot

@github-actions
Copy link
Copy Markdown
Contributor

🔥 Smoke Test: Copilot BYOK (Offline Mode)

Test Result
GitHub MCP (list_pull_requests) — latest merged: #2095 "[docs] docs: update --image-tag CLI reference for digest-aware format"
GitHub.com connectivity ✅ (pre-step unresolved, BYOK inference path active)
File write/read — /tmp/gh-aw/agent/smoke-test-copilot-byok-24643780249.txt
BYOK inference (agent → api-proxy → api.githubcopilot.com)

Running in BYOK offline mode (COPILOT_OFFLINE=true) via api-proxy → api.githubcopilot.com.

Overall: PASS — Author: @app/copilot-swe-agent · Assignees: @lpcox, @Copilot

🔑 BYOK report filed by Smoke Copilot BYOK

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[awf] Firewall: chatgpt.com blocked — codex plugin cache fails for gpt-5.3-codex

3 participants