fix: bring back too many recipients check
Needs https://github.com/nextcloud/mail/pull/9364
Fixes #6461
Anything missing except tests?
Anything missing except tests?
added tests
This checks for individuals but not expanded groups, right? As far as I can see we only expand at \OCA\Mail\Service\MailTransmission::sendLocalMessage so there is no warning for sending a message to one group of 1000 people.
This checks for individuals but not expanded groups, right? As far as I can see we only expand at
\OCA\Mail\Service\MailTransmission::sendLocalMessageso there is no warning for sending a message to one group of 1000 people.
That's true - I will add it.
Needs https://github.com/nextcloud/mail/pull/9364
We already have a recipients check but it depends on the Anti Abuse Protection to be turned on.
We can move the recipients check to be before we check for that to be enabled. Still needs the outbox status bc the exception won't bubble up so the mail will be sent regardless.
To Do:
Check if the recipient warning also expands groups (I don't think so), also needs a force option to send regardless.
Not worth rebasing.