Skip to content

fix(env): bind the WebSocket listener to the port nginx proxies [PE-92] - #495

Merged
kirychukyurii merged 1 commit into
mainfrom
fix/PE-92/env-defaults
Sep 7, 2026
Merged

kirychukyurii merged 1 commit into
mainfrom
fix/PE-92/env-defaults

Conversation

@kirychukyurii

Copy link
Copy Markdown
Contributor

WEBSOCKET shipped as :80 — the code default (flag:"websocket|:80|…"), which on a single-host install collides with nginx.

-WEBSOCKET=:80
+WEBSOCKET=127.0.0.1:10022

10022 is what the shipped nginx site already proxies to for /ws, /appointments/* and /endpoint/oauth2/* (nginx/default), and what the 25.08 unit passed as -websocket 127.0.0.1:10022.

PE-92.

🤖 Generated with Claude Code

WEBSOCKET defaulted to :80, which is the code default and collides with
nginx on a single-host install. The shipped nginx site proxies /ws,
/appointments and /endpoint/oauth2 to 127.0.0.1:10022.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@webitel-review

webitel-review Bot commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

🤖 Webitel Code Review

Зміна у файлі .env.example оновлює значення змінної оточення WEBSOCKET з :80 на 127.0.0.1:10022. Це безпечніше налаштування за замовчуванням для локальної розробки, оскільки воно обмежує доступ локальним хостом і не потребує привілейованого порту (80). Ризик зміни нульовий.

📋 Walkthrough (1 файл(и/ів))
Файл Зміни
.env.example Змінено адресу та порт WebSocket сервера за замовчуванням на 127.0.0.1:10022

Суттєвих зауважень немає. 👍


Index-grounded review across the Webitel codebase. Знахідки можуть бути неточними — перевіряйте перед застосуванням.

@kirychukyurii
kirychukyurii merged commit fef308b into main Sep 7, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant