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

Add (configurable) delay to sending notifications #1328

Open
hugalafutro opened this issue Jan 10, 2025 · 0 comments
Open

Add (configurable) delay to sending notifications #1328

hugalafutro opened this issue Jan 10, 2025 · 0 comments

Comments

@hugalafutro
Copy link

hugalafutro commented Jan 10, 2025

Description

I have several machines utilising diun, but few of them run so many containers it kills the telegram notifications with following error (to clarify, i.e. from 40 containers 20 update, I get notification for 15 and then 5 errors (numbers illustrational)):

diun  | Sat, 04 Jan 2025 12:01:20 GMT ERR Telegram notification failed error="failed to execute POST request to sendMessage: Post \"https://api.telegram.org/bot[REDACTED]/sendMessage\": context deadline exceeded" image=ghcr.io/sdr-enthusiasts/docker-piaware:latest

I propose a setting to delay each subsequesnt notification after 1st by x seconds to combat this. I am aware this might be telegram specific, but I'd wager other notif services won't enjoy spammy requests either.

Thanks for your consideration.

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

No branches or pull requests

1 participant