Skip to content

Add configurable bind host for TurboFieldfareServer - #79

Open
totiz wants to merge 3 commits into
drumih:mainfrom
totiz:feat/server-host-option
Open

Add configurable bind host for TurboFieldfareServer#79
totiz wants to merge 3 commits into
drumih:mainfrom
totiz:feat/server-host-option

Conversation

@totiz

@totiz totiz commented Aug 3, 2026

Copy link
Copy Markdown

Summary

  • Add --host so TurboFieldfareServer can bind beyond loopback (default remains 127.0.0.1).
  • Wire the flag through argument parsing into TurboFieldfareHTTPServer.start.
  • Document --host / 0.0.0.0 LAN bind in docs/OPENAI_SERVER.md, including the no-auth / no-TLS exposure warning.

Tested

  • swift build -c release --product TurboFieldfareServer — pass
  • Start with defaults and confirm ready URL uses 127.0.0.1 — pass (start(port:) binds 127.0.0.1; default --host parses to 127.0.0.1)
  • Start with --host 0.0.0.0 and confirm bind — pass (smoke bind + live server on *:8080, /health{"status":"ok"})
  • Reject empty --host (--host "" exits with usage error) — pass (exit 2, error: --host must not be empty)
  • Existing start(port:) server path still works (default host unchanged) — pass (loopback + 0.0.0.0 smoke via TurboFieldfareHTTPServer)

totiz added 2 commits August 4, 2026 04:37
Default remains 127.0.0.1; --host allows optional non-loopback binds such as 0.0.0.0.
Describe the default loopback bind, LAN exposure via 0.0.0.0, and the no-auth warning.
@totiz
totiz marked this pull request as ready for review August 3, 2026 22:00
@totiz totiz changed the title Add configurable bind host for TurboFieldfareServer tested: Add configurable bind host for TurboFieldfareServer Aug 3, 2026
@totiz totiz changed the title tested: Add configurable bind host for TurboFieldfareServer Add configurable bind host for TurboFieldfareServer Aug 3, 2026
Cover the default loopback host, successful parse of common bind values, and empty-host rejection.
@ARJSWork

ARJSWork commented Aug 5, 2026

Copy link
Copy Markdown

Any change this is merged soon? It would be great feature for the Server. I run ollama remote, so this would allow me to run fieldfare also remote on my mac mini m4.

@ulises-c

Copy link
Copy Markdown

This would be a great feature

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.

3 participants