Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Прикрутить stop_signal для websocket сервера #19

Open
nkiryanov opened this issue Jun 11, 2024 · 1 comment
Open
Assignees

Comments

@nkiryanov
Copy link
Collaborator

Отсюда #14

  1. Кажется, что тут тоже нужно прикрутить stop_signal? Дока тут .
@nkiryanov nkiryanov self-assigned this Jun 11, 2024
@nkiryanov
Copy link
Collaborator Author

Кажется, что тут тоже нужно прикрутить stop_signal? Дока тут .

@kazqvaizer кажись сейчас можно обойтись — ведь stop_signal будет передан в таски которые запущены в TaskGroup и как только какая-то из них остановится, то остановятся они все и сервис вебсокетов в том числе.

Но подумал, что можно чуть поменять подход и сделаю так:

  1. Уберу поддержку stop_signal у сервисов которые запущены асинхронно.
  2. Буду создавать TaskGroup в котором одна из задач будет чекать stop_signal

Так получится гибче — по stop_signal будут останавливаться все асинхронные задачи. Если их будет не 2, а условно 10, то stop_signal не потребуется передавать в каждую из них.

Но оставлю задачу в последнюю очередь — она плёвая, но время на неё до 14 вряд ли у меня найдётся.

@nkiryanov nkiryanov changed the title Прикурить stop_signal для websocket сервера Прикрутить stop_signal для websocket сервера Jun 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

1 participant