Skip to content

serve: wire Host allowlist through coli CLI - #649

Merged
JustVugg merged 1 commit into
JustVugg:devfrom
tasoo-oos:fix/coli-allowed-host-wiring
Jul 28, 2026
Merged

serve: wire Host allowlist through coli CLI#649
JustVugg merged 1 commit into
JustVugg:devfrom
tasoo-oos:fix/coli-allowed-host-wiring

Conversation

@tasoo-oos

Copy link
Copy Markdown

Summary

PR #606 added --allowed-host and COLI_ALLOWED_HOSTS to openai_server.py, but coli serve and coli web import serve() directly and bypass that parser. The documented primary entry points therefore could not configure the reverse-proxy Host allowlist.

Wire the existing allowlist through both CLI entry points, expose it in their help, and document the flag, environment variable, and reverse-proxy setup.

Validation

  • make -C c check
  • CUDA changes were tested with make -C c cuda-test (not applicable)
  • Performance claims include hardware, commands, and repeatable measurements (not applicable)

Compatibility

  • The default CPU build remains dependency-free
  • No model files, generated binaries, or benchmark artifacts are included

@tasoo-oos
tasoo-oos marked this pull request as ready for review July 27, 2026 09:45
@JustVugg
JustVugg merged commit 659dbd6 into JustVugg:dev Jul 28, 2026
@tasoo-oos
tasoo-oos deleted the fix/coli-allowed-host-wiring branch July 29, 2026 05:00
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.

2 participants