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

Отказоустойчивость для систем без модуля CONNECT #1059

Merged
merged 2 commits into from
Nov 18, 2022

Conversation

ai91
Copy link
Contributor

@ai91 ai91 commented Nov 17, 2022

Системы не использующие CONNECT страдают от проблемы с неработающим циклом connect. Что в свою очередь выливается в красный статус на дашборде.
Удаление модуля connect через менеджер модулей, в свою очередь, приводит к множественным рантайм-ошибкам на странице дашборда (из-за попытки модуля market обратиться к классу connect).

Данный фикс добавляет отказоустойчивость и устраняет ошибки дашборда для систем без модуля connect.

System without CONNECT module suffers from runtime errors on dashboard
Failsafety for system without CONNECT module
@Anisan
Copy link
Contributor

Anisan commented Nov 17, 2022

А как быть тем у кого есть купленные модули в CONNECT? Нужно предусмотреть настройку логина и пароля, если нет модуля CONNECT.

@ai91
Copy link
Contributor Author

ai91 commented Nov 17, 2022

А как быть тем у кого есть купленные модули в CONNECT? Нужно предусмотреть настройку логина и пароля, если нет модуля CONNECT.

Это будет уже другая фича, не связанная с данным фиксом. Либо не репортить красный статус при неработающем цикле, либо вводом у цикла признака деактивирован, вместо остановлен и выводом его из красного статуса на дашборде. Ну или вынос настроек логина и пароля из модуля CONNECT в общий конфиг. Что в свою очередь уже более глубокое изменение.

Данный же pull-request - именно багфикс поблемы с которой я столкнулся после весьма запоздалого обновления системы (года два не трогал работающий сервер, и вот).

@sergejey sergejey merged commit 5276214 into sergejey:alpha Nov 18, 2022
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.

3 participants