mbaechler commented on a change in pull request #833: URL: https://github.com/apache/james-project/pull/833#discussion_r782830357
########## File path: src/adr/0053-email-rate-limiting.md ########## @@ -0,0 +1,65 @@ +# 53. Email rate limiting + +Date: 2021-01-10 + +## Status + +Accepted (lazy consensus). + +Not yet implemented. + +## Context + +Rate limiting is one of the common features expected from an email system. Examples: SaaS is +one https://www.fastmail.help/hc/en-us/articles/1500000277382-Account-limits#sending/receiving +, https://support.google.com/mail/answer/22839 + +They limit how many emails users can send/receive from/to each email account over a given period of time. +We believe the rate-limiting will help James to have more benefits: + +- Control of the resource +- Easy to dynamic config the user policy. +- Complements the storage quota +- Can be a security requirement for SaaS deployments. +- Minimise impacts of Open-relay types of compression. +- Limiting the amount of email sent to third parties can also prevent them from considering you as an open relay and can + be beneficial as well. + +## Decision + +Set up a new maven project dedicated to rating limiting. This allows the rate limiting mailets to be embedded in a James +server as a soft dependency using the external-jar loading mechanism. Please note that this will take the form of an +extension jars, that could be dropped in one's James installation, and thus is optional, and not of a runtime +dependency. + +Rate limiting will be enabled per sender, per receiver and globally. For each we will provide options for email size and +email count. + +- This can be done via mailets: + - PerSenderRateLimit is per sender + - PerRecipientRateLimit is per recipient + - GlobalRateLimit for everyone Depending on the position in the pipeline this could allow rating limit all emails in + transit, relayed emails or locally delivered emails. + The rate limit will be config + in [mailetcontainer.xml](/server/apps/distributed-app/sample-configuration/mailetcontainer.xml). + +- Those mailets will be based on a generic RateLimiter interface. We will propose two implementations for it: + - In memory (guava based) suitable for single instance deployments + - [Redis](https://redis.io) based, suitable for distributed deployments. Review comment: Yes. Anyway, I was just curious about the Redis choice, I have a very bad experience performance-wise with Redis. Cassandra is probably not better for that. And I think that an in-memory solution fits the need for the reason you described, too. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
