Skip to content

Telegram output#1

Open
openbsod wants to merge 2 commits into
orgtechservice:masterfrom
openbsod:master
Open

Telegram output#1
openbsod wants to merge 2 commits into
orgtechservice:masterfrom
openbsod:master

Conversation

@openbsod

@openbsod openbsod commented Apr 9, 2018

Copy link
Copy Markdown

Привет.

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

@WST

WST commented Apr 9, 2018

Copy link
Copy Markdown
Member

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

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

WST commented Apr 10, 2018

Copy link
Copy Markdown
Member

Хардкод конфигурационных данных в скрипт — не есть хорошо.
Отправка через 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

@WST WST left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

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

@WST

WST commented Apr 11, 2018

Copy link
Copy Markdown
Member

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

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