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

notify_telegram plugin #4303

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

YanikusGG
Copy link

@YanikusGG YanikusGG commented Apr 26, 2024

This is initial implementation for notify_telegram plugin. Later I will create PR for collectd 6 also.

The notify_telegram plugin uses libcurl and Telegram Bot API to send notifications to a configured telegram users.
It also can send config instruction with ChatID in response to all users by long polling or webhooks using libcurl, libyajl, libmicrohttpd.

ChangeLog: Introduce notify_telegram plugin

Reference: https://core.telegram.org/bots/api

@YanikusGG YanikusGG requested a review from a team as a code owner April 26, 2024 03:47
@matwey matwey added the Feature label Apr 26, 2024
src/collectd.conf.pod Outdated Show resolved Hide resolved
src/collectd.conf.pod Outdated Show resolved Hide resolved
src/collectd.conf.pod Outdated Show resolved Hide resolved
src/collectd.conf.pod Outdated Show resolved Hide resolved
src/notify_telegram.c Outdated Show resolved Hide resolved
src/notify_telegram.c Outdated Show resolved Hide resolved
src/notify_telegram.c Outdated Show resolved Hide resolved
src/notify_telegram.c Outdated Show resolved Hide resolved
src/notify_telegram.c Outdated Show resolved Hide resolved
src/notify_telegram.c Outdated Show resolved Hide resolved
@eero-t
Copy link
Contributor

eero-t commented Apr 26, 2024

This is initial implementation for notify_telegram plugin. Later I will create PR for collectd 6 also.

New plugins should go to collectd-6.0 branch, only existing plugins are updated in v5.x code (main) , but please handle review feedback first.

@YanikusGG
Copy link
Author

This is initial implementation for notify_telegram plugin. Later I will create PR for collectd 6 also.

New plugins should go to collectd-6.0 branch, only existing plugins are updated in v5.x code (main) , but please handle review feedback first.

Thank you for your helpful issues. I've fixed most of them.

@eero-t
Copy link
Contributor

eero-t commented May 13, 2024

Thanks, fixes look good! Next step is review from Florian (@octo) for the v6 version of the PR.

(He hasn't been active for a while, but hopefully you don't need to wait too long...)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants