Skip to content

fix: bind to 127.0.0.1 by default, add --host option - #13

Open
saherr1969 wants to merge 1 commit into
pajaydev:mainfrom
saherr1969:fix/default-localhost-binding
Open

fix: bind to 127.0.0.1 by default, add --host option#13
saherr1969 wants to merge 1 commit into
pajaydev:mainfrom
saherr1969:fix/default-localhost-binding

Conversation

@saherr1969

Copy link
Copy Markdown
Contributor

Fixes #11.

Previously the server listened on 0.0.0.0 (all interfaces) by default, exposing conversation history to the local network without the user's knowledge. This change defaults the listen address to 127.0.0.1 (localhost only) so the server is not accessible from other devices.

Users who want network access can opt in with:

kiro-history --host 0.0.0.0

Changes:

  • server/index.ts: startServer() accepts a hostname parameter, defaults to '127.0.0.1', passes it to @hono/node-server serve().
  • server/cli.ts: Added --host CLI option (default: 127.0.0.1).
  • README.md: Documented the --host option in the options table and added a usage example.

Fixes pajaydev#11.

Previously the server listened on 0.0.0.0 (all interfaces) by default,
exposing conversation history to the local network without the user's
knowledge. This change defaults the listen address to 127.0.0.1
(localhost only) so the server is not accessible from other devices.

Users who want network access can opt in with:

  kiro-history --host 0.0.0.0

Changes:
- server/index.ts: startServer() accepts a hostname parameter, defaults
  to '127.0.0.1', passes it to @hono/node-server serve().
- server/cli.ts: Added --host <address> CLI option (default: 127.0.0.1).
- README.md: Documented the --host option in the options table and added
  a usage example.
@saherr1969

Copy link
Copy Markdown
Contributor Author

Hi @pajaydev — just checking in on this. Let me know if you'd like any changes. Thanks!

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.

Listening Address Binding

1 participant