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

Limit email rate and batch the accumulated notifications into one message #2500

Closed
1 task done
gitwildnow opened this issue Dec 29, 2022 · 4 comments
Closed
1 task done
Labels
area:notifications Everything related to notifications feature-request Request for new features to be added

Comments

@gitwildnow
Copy link

⚠️ Please verify that this feature request has NOT been suggested before.

  • I checked and didn't find similar feature request

🏷️ Feature Request Type

Other

🔖 Feature description

Add the option to batch notifications into one email message. Right now if my router goes down I get 33 emails about things being down and then 33 emails about them coming online again - uggh.

I'm looking for a simple batch-time setting. If I tell it to batch for five minutes then when a notification gets queued the timer starts. All the notifications that happen in my five minute window get queued. When the window is done, one email gets sent with all applicable notifications mentioned in the body of the message.

Naturally this delays notification, so set the batch time to something that makes sense for your purposes.

✔️ Solution

In the Settings->Notifications set the Batch Time to however many minutes you like. Default is zero, consistent with current behavior.

❓ Alternatives

No response

📝 Additional Context

Watching email for uptime kuma notifications.

@gitwildnow gitwildnow added the feature-request Request for new features to be added label Dec 29, 2022
@Robin-Sch
Copy link

Or maybe have a way to "group" applications. If one goes down, immediately check the rest?? and then send the batch notification.

@CommanderStorm
Copy link
Collaborator

CommanderStorm commented Aug 16, 2023

@gitwildnow
I think this is a duplicate of #1137
If you agree, could you please close this Issue, as duplicates only create immortal zombies and are really hard to issue-manage?
If not, what makes this issue unique enough to require an additional issue? (Could this be integrated into the issue linked above?) ^^

@Robin-Sch
Copy link

Robin-Sch commented Aug 20, 2023

This issue: sending the "service went down" notifications in a batch. Having a setting which specifies the time limit how notifications should be "batched" (for example, setting it to 5 minutes will "batch" together if 2 services go down in those 5 minutes).
#1137: sending a notification with all services which are down. This also requires a settings/timeout like this issue I guess... not sure

  • additionally, this issue adds the ability to disable/enable both down and up notifications, so you can only get for example a notification when a service is down and not when a service is back up again.

Let's say yesterday service x went down (and has not come back up), and now service y and z went down.

This issue:
Notification yesterday: service x went down
Notification today: service y and z went down

#1137:
Notification yesterday: service x is down
Notification today: service x, y and z are down

Personally, I'd say keep #1137 open, but add the setting so we can specify a custom timeout. Like for example, if a service goes down, wait a minute and see if more go down, if not, send the notification that only that service is down. If multiple services go down, send the list with all services which went down.
I also like that idea of getting a "list" with services which are down, instead of just a notification that service x went down.

@CommanderStorm CommanderStorm added the area:notifications Everything related to notifications label Dec 4, 2023
@CommanderStorm
Copy link
Collaborator

@gitwildnow
We are consolidating duplicate issues a bit to make issue management easier.
I think, we should track this issue in #1137 as there is no functional difference (both issues target the ability to group multiple notifications)
⇒ I am going to close this as a duplicate.

@CommanderStorm CommanderStorm closed this as not planned Won't fix, can't repro, duplicate, stale Dec 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:notifications Everything related to notifications feature-request Request for new features to be added
Projects
None yet
Development

No branches or pull requests

3 participants