Skip to content

[codex] Fix Browser plugin file URL policy#306

Merged
ilysenko merged 1 commit into
mainfrom
codex/fix-browser-file-url-policy
May 24, 2026
Merged

[codex] Fix Browser plugin file URL policy#306
ilysenko merged 1 commit into
mainfrom
codex/fix-browser-file-url-policy

Conversation

@ilysenko
Copy link
Copy Markdown
Owner

Summary

  • Patch the staged Browser plugin URL policy so local file:// targets are allowed on Linux.
  • Keep the policy narrow: data:, javascript:, ftp:, invalid URLs, and remote file://host/... URLs stay blocked.
  • Update Browser plugin staging smoke fixtures to match the current upstream try { new URL(...) } catch { return false } helper shape.

Fixes #268.

Root cause

The bundled Browser plugin advertises local file:// support, and the downstream approval flow already knows how to request approval for file: origins. The earlier URL policy gate rejected file: before that approval path could run.

Validation

  • bash -n scripts/lib/bundled-plugins.sh tests/scripts_smoke.sh
  • git diff --check
  • tests/scripts_smoke.sh
  • make build-app-fresh against freshly downloaded upstream DMG 26.519.41501 / Electron 42.1.0 / SHA256 3101c05dfd40314115418c4ad87edee02419d097f7164c677ddbdd4559ad8a42
  • Verified the staged Browser client contains codexLinuxFileUrlPolicy in the real upstream helper and no URL-policy insertion warning is emitted

@ilysenko ilysenko marked this pull request as ready for review May 24, 2026 19:37
@ilysenko ilysenko merged commit 0850ed9 into main May 24, 2026
5 checks passed
@ilysenko ilysenko deleted the codex/fix-browser-file-url-policy branch May 24, 2026 19:37
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.

Browser plugin file:// support is blocked by URL policy on Linux desktop build

1 participant