Fix exception handling #33
Conversation
| except (ConnectionError, ConnectionRefusedError): | ||
| logging.error(f"Cannot connect to broker {settings.BROKER_URL}") | ||
| sys.exit(1) | ||
| except Exception as e: | ||
| logging.error(e) | ||
| sys.exit(1) |
There was a problem hiding this comment.
кстати о гайдлайнах, про это мы не писали, а можно сделать сразу так, чтобы у нас был стандарт на оформление сервисов по-правильному (я так понимаю, тут ещё надо будет pylint урезонить, чтобы он не ругался на ловлю Exception)
| except subprocess.TimeoutExpired as e: | ||
| raise ValueError("Timeout expired: " + str(e)) |
There was a problem hiding this comment.
не очень понятно, зачем это заворачивать в ValueError
| except subprocess.CalledProcessError as e: | ||
| raise ValueError("Error during request: " + str(e)) |
There was a problem hiding this comment.
а зачем это заворачивать в ValueError?
| if hasattr(options, "func"): | ||
| mqtt.start() | ||
| return options.func(options, settings, mqtt) | ||
| try: |
There was a problem hiding this comment.
резко всю функцию сдвинули, при этом не покрыли обработкой исключений всё то, что происходит выше (а там ведь и файлы читаются, и клиент создаётся, как будто бы тоже можно исключений наловить, ещё и таких, чтобы systemd не перезапускал это потом, а говорил, что сервис не сконфигурирован как надо)
There was a problem hiding this comment.
как будто бы тоже можно исключений наловить, ещё и таких, чтобы systemd не перезапускал это потом, а говорил, что сервис не сконфигурирован как надо
Про это отдельно тут #38
No description provided.