Skip to content

Bypass proxy for CLI health checks#858

Open
MKuBMax wants to merge 1 commit intojundot:mainfrom
MKuBMax:fix/cli-health-check-no-proxy
Open

Bypass proxy for CLI health checks#858
MKuBMax wants to merge 1 commit intojundot:mainfrom
MKuBMax:fix/cli-health-check-no-proxy

Conversation

@MKuBMax
Copy link
Copy Markdown
Contributor

@MKuBMax MKuBMax commented Apr 19, 2026

This extends the existing no-proxy fix to omlx.cli.launch_command so local health checks do not inherit system proxy settings.

Changes:

  • Use requests.Session(trust_env=False) for the launch health probe in omly/cli.py
  • Add a regression test that asserts the health check uses a session with trust_env disabled

Validation:

  • uv run pytest -q tests/test_cli.py

@jundot jundot force-pushed the main branch 2 times, most recently from 7844f15 to b078330 Compare April 28, 2026 02:11
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.

1 participant