Kick Member instead of closing Room on internal errors (#27)

[alexlapa]

Synopsis

На ранних этап развития проекта во многих непонятных ситуациях мы просто закрывали комнату. Тогда еще было не ясно как обрабатывать определенные некорректные ситуации, а, таким образом их было проще обнаруживать.

Постепенно таких моментов становилось все меньше, на данный момент осталось всего два.

Необходимо их убрать, и ввести правило, что, комната может закрываться только в двух случаях:

1. По команде через Control API
2. В случае завершения приложения.

Solution

Что за два места, которые нужно поправить:

1. Отправка PeersRemoved Event'а. Просто убираем shutdown, все проблемы рассинхронов состояния в результате утерянных сообщений чинятся в #47.
2. Ошибка обработки комманды. Эти ошибки на практике не происходят, следовательно, накручивать логику их обработки нецелесообразно, так как это потребует значительных временных ресурсов, а выхлоп стремится к нулю. Также, нельзя исключать возможность работы с модифицированным клиентом (например злоумышленно модифицированным). Простейшим выходом из этой ситуации будет отключение клиента. Клиент будет отключен с CloseReason::InternalError, в CallbackEvent'е будет значится причина "Kicked".

Checklist

• Created PR:
  • [x] In draft mode
  • [x] Name contains Draft: prefix
  • [x] Name contains issue reference
  • [x] Has k:: labels applied
  • [x] Has assignee
• [x] Documentation is updated (if required)
• [x] Tests are updated (if required)
• [x] Changes conform code style
• [x] CHANGELOG entry is added (if required)
• [x] FCM (final commit message) is posted
  • [x] and approved
• [x] Review is completed and changes are approved
• Before merge:
  • [x] Milestone is set
  • [x] PR's name and description are correct and up-to-date
  • [x] Draft: prefix is removed
  • [x] All temporary labels are removed

[alexlapa]

FCM:

Kick Member instead of closing Room on internal errors (#175, #27)

Additionally:
- update Cargo deps