Skip to content

Conversation

@openbsod
Copy link

@openbsod openbsod commented Apr 9, 2018

Привет.

Прилагаю дополнение скрипта rkn-check.py для отправки результатов проверки в Телеграм.

@WST
Copy link
Member

WST commented Apr 9, 2018

Добрый день, спасибо за интерес, завтра посмотрим, т.к рабочий день подходит к концу.

@WST WST self-requested a review April 10, 2018 07:00
@WST WST self-assigned this Apr 10, 2018
@WST
Copy link
Member

WST commented Apr 10, 2018

Хардкод конфигурационных данных в скрипт — не есть хорошо.
Отправка через curl, честно говоря, мне тоже не очень нравится.
Есть пакет python-telegram-bot, позволяющий легко отправить сообщение следующим образом:

from telegram import Bot
bot = Bot('*********************')
bot.send_message(CHAT_ID, "Hello, world!")

Но у этого подхода есть очевидный недостаток — данный Python-пакет недоступен в репозиториях Linux Mint / Debian / Ubuntu, а исходная идея проекта в том, чтобы все зависимости легко ставились через apt-get.

Поэтому, пожалуйста, сделайте следующее:
— ID чата и ключ API Телеграма нужно вынести в конфигурационный файл
— Если задействовать python-telegram-bot, то обернуть его использование в try…except, чтобы скрипт мог работать в случае, если данный пакет не установлен в системе
— Если задействовать curl, то нужно добавить зависимость от curl в файл DEBIAN/control

Copy link
Member

@WST WST left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Описал в комментарии

@WST
Copy link
Member

WST commented Apr 11, 2018

И, кстати да, совсем забыл добавить, что саму опцию отправки неплохо бы сделать отключаемой.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants