Skip to content

feat: added optional parameter to specify which host interface to listen on when using http transport#419

Open
thrasher wants to merge 4 commits intotobi:mainfrom
thrasher:main
Open

feat: added optional parameter to specify which host interface to listen on when using http transport#419
thrasher wants to merge 4 commits intotobi:mainfrom
thrasher:main

Conversation

@thrasher
Copy link
Copy Markdown

Treatment of the hard-coded "localhost" interface can vary by machine environment. This causes a problem specifically when trying to connect from a Docker service (ex: Open-WebUI via MCP) to QMD using host.docker.internal to localhost - it fails to connect. Specifying the 0.0.0.0 interface allows the connection, but alternatives should also be possible.

The --host parameter is optional, and defaults to the original behavior. Documentation and tests were added to support the change, while preserving the style. All tests pass.

Jason Thrasher added 3 commits March 16, 2026 20:33
…default host of "localhost" causes problems on some systems when connecting from a Docker container if not bound to 0.0.0.0. Specifying the specific host option allows this behavior, while still utilizing the default localhost as a fallback.

Updated docs, and passes all tests that expect the default to be localhost. Currently tests don't try an alternative host.
@thrasher thrasher marked this pull request as ready for review March 16, 2026 20:14
@thrasher thrasher marked this pull request as draft March 16, 2026 20:50
…sing the declared option types, causing the npm run build to fail. Also found missing host parameter not realized until running it.
@thrasher thrasher closed this Mar 16, 2026
@thrasher thrasher reopened this Mar 16, 2026
@thrasher thrasher marked this pull request as ready for review March 16, 2026 21: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