vvzvlad/gitmost
1
0
Fork 0
Code Issues 32 Pull Requests 10 Packages Projects Releases Activity

fix(health): утечка ioredis-клиента в /health при недоступном Redis [T0.2] #483

New Issue
Closed
opened 2026-07-11 01:51:26 +03:00 by agent_vscode · 1 comment
agent_vscode commented 2026-07-11 01:51:26 +03:00
Collaborator
Copy Link
Copy Source

Суть. apps/server/src/integrations/health/redis.health.ts (pingCheck): new Redis(...) на каждый вызов; redis.disconnect() только на success-пути. При недоступном Redis каждый вызов /health навсегда добавляет реконнектящийся клиент (maxRetriesPerRequest: 15 не останавливает реконнекты самого соединения).

Сценарий отказа. Redis упал → docker/k8s-проба долбит /health каждые N секунд → сотни живых клиентов с таймерами реконнекта → рост памяти/CPU, лавина попыток соединений усугубляет инцидент.

Правка.

  1. Минимум: try/finally с disconnect().
  2. Правильно: один долгоживущий клиент для health-проб (lazyConnect, maxRetriesPerRequest: 1).
  3. Системно: единая фабрика Redis-подключений с регистрацией в shutdown-хуке — закрывает класс, который уже чинили точечно (#254, #256).

Тест. Интеграционный — N вызовов pingCheck при лежащем Redis не увеличивают число живых хэндлов (наблюдаемое свойство, не мок).

Зависимости. Нет — независимый мелкий PR.

**Суть.** `apps/server/src/integrations/health/redis.health.ts` (`pingCheck`): `new Redis(...)` на каждый вызов; `redis.disconnect()` только на success-пути. При недоступном Redis каждый вызов `/health` навсегда добавляет реконнектящийся клиент (`maxRetriesPerRequest: 15` не останавливает реконнекты самого соединения). **Сценарий отказа.** Redis упал → docker/k8s-проба долбит `/health` каждые N секунд → сотни живых клиентов с таймерами реконнекта → рост памяти/CPU, лавина попыток соединений усугубляет инцидент. **Правка.** 1. Минимум: `try/finally` с `disconnect()`. 2. Правильно: один долгоживущий клиент для health-проб (`lazyConnect`, `maxRetriesPerRequest: 1`). 3. Системно: единая фабрика Redis-подключений с регистрацией в shutdown-хуке — закрывает класс, который уже чинили точечно (#254, #256). **Тест.** Интеграционный — N вызовов pingCheck при лежащем Redis не увеличивают число живых хэндлов (наблюдаемое свойство, не мок). **Зависимости.** Нет — независимый мелкий PR.
agent_vscode commented 2026-07-11 02:00:02 +03:00
Author
Collaborator
Copy Link
Copy Source

Заменено итерационным ишью #486 (коммит 2). Закрыто как дубль.

Заменено итерационным ишью #486 (коммит 2). Закрыто как дубль.
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
epic
Large multi-phase effort spanning many changes
 needs-human
эскалация: нужно решение человека
 review/approved
в последнем ревью нет открытых blocking-находок
 review/changes-requested
последнее ревью оставило открытые blocking-находки
 review/needs
head не ревьюился (head != reviewed_head)
 возможно, баг
Похоже на баг, но требует подтверждения/воспроизведения; чиним, если повторится
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
agent_coder agent_qa agent_reviewer agent_vscode vvzvlad
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: vvzvlad/gitmost#483
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?