Skip to content

feat: (config) improve web & management configuration - #119

Open
anzz1 wants to merge 4 commits into
LostCityRS:274from
anzz1:patch-1
Open

feat: (config) improve web & management configuration#119
anzz1 wants to merge 4 commits into
LostCityRS:274from
anzz1:patch-1

Conversation

@anzz1

@anzz1 anzz1 commented Sep 2, 2026

Copy link
Copy Markdown
  • Put the management server/service under its own configuration header, make it possible to enable/disable it

  • Allow configuring what interface the web and management services are bound to
    For example, you could bind management to 127.0.0.1 (localhost) so setup/prometheus aren't accessible from outside internet, you could ingest prometheus stats locally or put it behind nginx reverse proxy or whatever

  • Fix app.ts easyStartup not respecting the login/friend/logger enabled configuration option.

  • Add configuration options for chat logging

@anzz1
anzz1 marked this pull request as draft September 2, 2026 18:34
@anzz1

anzz1 commented Sep 2, 2026

Copy link
Copy Markdown
Author

this isn't actually working yet, i meant to only bind the web server, not the web socket server. 

edit: never mind, it works fine. just needed to properly pass the websocket, like this in nginx

location / {
    proxy_pass        http://localhost:8888/;
    proxy_set_header  Upgrade $http_upgrade;
    proxy_set_header  Connection "upgrade";
}

@anzz1
anzz1 marked this pull request as ready for review September 2, 2026 18:38
@anzz1 anzz1 changed the title allow configuring what interface web server is bound to feat: allow configuring what interface web server is bound to Sep 3, 2026
@anzz1 anzz1 changed the title feat: allow configuring what interface web server is bound to feat: improve web & management configuration Sep 3, 2026
@anzz1 anzz1 changed the title feat: improve web & management configuration feat: (config) improve web & management configuration Sep 3, 2026
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