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 polling protocol environment #561

Closed
wounkun opened this issue Sep 26, 2022 · 0 comments · Fixed by #2250
Closed

telegram polling protocol environment #561

wounkun opened this issue Sep 26, 2022 · 0 comments · Fixed by #2250
Labels
feature request New feature or request

Comments

@wounkun
Copy link

wounkun commented Sep 26, 2022

As I noticed before about private server where running Grafana OnCall need use ChatOps/Alert notifications through webhook with self-signed certificate (#530), but what about polling protocol?

I think if we get telegram connections through polling protocol it will be more useful than we trying connect by webhook from private servers. In results there's no need to open external port for receiving messages from telegram bot. :)

@Matvey-Kuk Matvey-Kuk added the feature request New feature or request label Sep 27, 2022
@alexintech alexintech mentioned this issue Jun 15, 2023
3 tasks
joeyorlando pushed a commit that referenced this issue Aug 24, 2023
# What this PR does

Runs Telegram long polling to get updates. 
It's enabled by setting `FEATURE_TELEGRAM_LONG_POLLING_ENABLED=True`.
That will disable webhook and run separate deployment for telegram long
polling.

Telegram long polling is not very HA mode, but it does not need to
expose webhook url to internet and simplifies telegram integration.

## Which issue(s) this PR fixes

closes #561 

## Checklist

- [x] Unit, integration, and e2e (if applicable) tests updated
- [x] Documentation added (or `pr:no public docs` PR label added if not
required)
- [x] `CHANGELOG.md` updated (or `pr:no changelog` PR label added if not
required)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants