AWS SES Transport fails on sending a batch immediately

[mabumusa1]

Bug Description

The more details the better...

Q	A
Mautic version	3.2.5
PHP version	7.4
Browser

Steps to reproduce

1. Make sure you are sending using immediate mode
2. Create a segment with more than 50 contacts
3. Send an email to that segment
4. The email will be failed to send
5. There is a related error for throttling (logs below)

Log errors

[2021-02-03 08:01:33] mautic.ERROR: [MAIL ERROR] Token amount (51) is larger than the capacity (3). (send); {"exception":"[object] (Swift_TransportException(code: 0): Token amount (51) is larger than the capacity (3). at app/bundles/EmailBundle/Swiftmailer/Transport/AbstractTokenArrayTransport.php:300)"} []

[2021-02-05 08:25:00] mautic.ERROR: [MAIL ERROR] Expected response code 250 but got code "454", with message "454 Throttling failure: Maximum sending rate exceeded. " (send); email@domain.com {"exception":"[object] (Swift_TransportException(code: 454): Expected response code 250 but got code \"454\", with message \"454 Throttling failure: Maximum sending rate exceeded.\r\n\" at /home/smartconnectamba/public_html/vendor/swiftmailer/swiftmailer/lib/classes/Swift/Transport/AbstractSmtpTransport.php:457)"} []

Solution to be implemented must look into the following lines:

        if (!$this->started) {
            $this->amazonClient  = $this->createAmazonClient();

            $account             = $this->amazonClient->getAccount();
            $emailQuotaRemaining = $account->get('SendQuota')['Max24HourSend'] - $account->get('SendQuota')['SentLast24Hours'];

            if (!$account->get('SendingEnabled')) {
                $this->logger->error('Your AWS SES is not enabled for sending');
                throw new \Exception('Your AWS SES is not enabled for sending');
            }

            if (!$account->get('ProductionAccessEnabled')) {
                $this->logger->info('Your AWS SES is in sandbox mode, consider moving it to production state');
            }

            if ($emailQuotaRemaining <= 0) {
                $this->logger->error('Your AWS SES quota is currently exceeded, used '.$account->get('SentLast24Hours').' of '.$account->get('Max24HourSend'));
                throw new \Exception('Your AWS SES quota is currently exceeded');
            }

            $this->concurrency   = floor($account->get('SendQuota')['MaxSendRate']);

            $this->started = true;
        }

[RCheesley]

Thanks for flagging up this issue @mabumusa1 - are you able to work on this or should we find someone to fix this?

[mabumusa1]

@RCheesley I am working on a solution for it.

[RCheesley]

Amazing - thanks for letting us know!

[mabumusa1]

Thanks for flagging up this issue @mabumusa1 - are you able to work on this or should we find someone to fix this?

The fix is on the attached PR

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[stale]

This issue has been automatically closed because it has not had recent activity. If the reported issue persists, please create a new issue and link back to this one for reference. Thank you for your contributions.

[stale]

This issue or PR has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. If you would like to keep it open please let us know by replying and confirming that this is still relevant to the latest version of Mautic and we will try to get to it as soon as we can. Thank you for your contributions.

[stale]

This issue or PR has been automatically closed because it has not had recent activity. In the case of issues, if it persists in the latest version of Mautic, please create a new issue and link back to this one for reference. With PRs if you wish to pick up the PR and update it so that it can be considered for a future release, please comment and we will re-open it. Thank you for your contributions.

[github-actions]

⚠️COMMENT VISIBILITY WARNING⚠️

Comments on closed issues are hard for our team to see.
If this issue is continuing with the lastest stable version of Mautic, please open a new issue that references this one.