Skip to content

feat: add host parameter to allow listening on a specific ip address#216

Merged
perber merged 1 commit intomainfrom
feature/allow-binding-to-a-local-address
Oct 19, 2025
Merged

feat: add host parameter to allow listening on a specific ip address#216
perber merged 1 commit intomainfrom
feature/allow-binding-to-a-local-address

Conversation

@perber
Copy link
Owner

@perber perber commented Oct 19, 2025

No description provided.

Copilot AI review requested due to automatic review settings October 19, 2025 13:49
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds a host parameter to LeafWiki to allow configuring the IP address the server binds to, enabling deployment scenarios where the server should only listen on specific interfaces (e.g., localhost behind a reverse proxy).

  • Added --host CLI flag and LEAFWIKI_HOST environment variable with default value 0.0.0.0
  • Updated server startup to bind to the specified host and port combination
  • Added documentation explaining the new parameter and its use case with reverse proxies

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
readme.md Added documentation for the new host parameter in CLI flags, environment variables, and usage examples
cmd/leafwiki/main.go Implemented host parameter handling and updated server binding logic

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@perber perber merged commit 7af5b8f into main Oct 19, 2025
4 checks passed
@perber perber linked an issue Oct 19, 2025 that may be closed by this pull request
@perber perber deleted the feature/allow-binding-to-a-local-address branch October 23, 2025 17:54
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.

Allowing binding to a local address

2 participants