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

Telegram output #1

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Telegram output #1

wants to merge 2 commits into from

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