Skip to content

fix(e2e): add uv-managed python binary glob to forward proxy L7 test#686

Merged
johntmyers merged 1 commit intomainfrom
fix/forward-proxy-l7-test-binary-glob
Mar 30, 2026
Merged

fix(e2e): add uv-managed python binary glob to forward proxy L7 test#686
johntmyers merged 1 commit intomainfrom
fix/forward-proxy-l7-test-binary-glob

Conversation

@johntmyers
Copy link
Copy Markdown
Collaborator

Summary

  • Fixes the forward_proxy_allows_l7_permitted_request e2e test failure introduced by fix(proxy): add L7 inspection to forward proxy path #666
  • The base sandbox image installs Python via uv at /sandbox/.uv/python/*/bin/python*, but the proxy resolves binary identity via /proc/PID/exe (the real path, not the symlink at /usr/local/bin/python3)
  • The test policy only listed /usr/bin/python* and /usr/local/bin/python*, so OPA denied at L4 before L7 evaluation could run

Related Issue

Fixes e2e failure from #666

Changes

  • Added /sandbox/.uv/python/*/bin/python* to the binary allowlist in the forward proxy L7 bypass test policy

Testing

  • Verified locally: both forward_proxy_allows_l7_permitted_request and forward_proxy_denies_l7_blocked_request pass

The base sandbox image installs Python via uv at
/sandbox/.uv/python/*/bin/python*, but the proxy resolves binary
identity via /proc/PID/exe (the real path, not the symlink). The test
policy only listed /usr/bin/python* and /usr/local/bin/python*, so OPA
denied the connection at L4 before L7 evaluation could run.
@johntmyers johntmyers requested a review from a team as a code owner March 30, 2026 18:45
@johntmyers johntmyers self-assigned this Mar 30, 2026
@johntmyers johntmyers added the test:e2e Requires end-to-end coverage label Mar 30, 2026
@johntmyers johntmyers merged commit 0832f11 into main Mar 30, 2026
12 of 13 checks passed
@johntmyers johntmyers deleted the fix/forward-proxy-l7-test-binary-glob branch March 30, 2026 18:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test:e2e Requires end-to-end coverage

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant