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

Wishlist/feature request - reduce number of emails sent to users #36

Open
aputtu opened this issue Mar 19, 2024 · 2 comments
Open

Wishlist/feature request - reduce number of emails sent to users #36

aputtu opened this issue Mar 19, 2024 · 2 comments
Labels
documentation Improvements or additions to documentation enhancement New feature or request

Comments

@aputtu
Copy link

aputtu commented Mar 19, 2024

We receive reports (id 32634) on occasions, where users get overwhelmed by the number of emails received.
There seems to be a use for:

  1. Alerts regarding security issues.
  2. Warnings regarding various areas of concern.
  3. Information on file changes and similar functional usage.

In addition there are the type of emails, where e.g. the system will send out mails when a connection fails or similar. We will want to warn users at time, but (most often) not have them receiving excessive amount of identical warnings.

Questions to raise:

  • Can we rate-limit or compound outgoing mails, so we don't send out excessive number of identical alerts/warning.
  • Would it make sense to make built-in information on activity instead of sending emails?
  • Option to opt-out of certain types of emails

Not sure which part of the issues that belong to Migrid development and which part that belongs to server administrator.

@jonasbardino jonasbardino added documentation Improvements or additions to documentation enhancement New feature or request labels Apr 25, 2024
@jonasbardino
Copy link
Contributor

I'm pretty sure 99% of these reports are about the built in notifications system warning about valid issues like repeated failed SFTP/WebDAVS login attempts and often the resulting case of hitting the login rate limit. Such emails are typically triggered when a client has set up an SFTP or WebDAVS network drive with automatic retry - and it keeps trying login despite repeat errors. Common service login error causes include when main user account expired (ERDA/SIF FAQ) or when the required twofactor session expired. The latter is mandatory for GDP sites like SIF and optional on general sites like ERDA.

The notification system already does internal batching to avoid sending an email on every error, but if the client keeps failing login it will result in additional emails every once in a while until the client stops hammering. This batching window may be adjusted but a number of these messages are about e.g. expired main account due to lack of web activity, so it does not make sense to inform about it on web. We really only can email or completely suspend the account to get user attention in such cases.

Apart from perhaps improving the actual warnings emails and related documentation I think @Rehr is best acquainted with the notification code and any frequency adjustments.

@Martin-Rehr
Copy link
Contributor

The notification interval is hard coded to 60 seconds and was chosen to give users a quick response upon failed logins. Similar errors are batched within the interval.

We could extend the system to suppress similar errors that occurs across notification intervals but I'll say this goes at the end of the nice-to-have list, since users are only bothered if one of their own clients keeps hammering on the system.

Last but not least we could use fail2ban to block users with clients "on the loose".

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants