[JAMES-881] ValidSentAddresses not removed from recipients list on SendFailedException

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Won't Fix
• Affects Version/s: 2.3.0, 2.3.1
• Component/s: Remote Delivery
• Assignee: Unassigned
• Reporter: Jean Marc
• Created: 17/Nov/08 18:37
• Updated: 09/May/22 02:06
• Resolved: 09/May/22 02:06

Description

In the deliver() loop of RemoteDelivery, when a SendFailedException is received as a "452 Too Many Recipients" from a remote server, the processing continues to the next available server regardless of the ValidSentAddresses, and the whole list of original recipients is sent again to the next server of the domain, creating multiple duplicate emails in the recipients accounts.

The typical example is Yahoo, which accepts a variable number of recipients per email from a specific server according to their internal business rules (bulk settings, SPF, reverse DNS, etc). For example, Yahoo servers receive a mail for 69 addresses, accepts the email for the first 30, then stops processing the mail and returns code=452 and the list of valid sent/unsent addresses. James logs the list of valid sent, and continues to the next mx record, sending the email to the 69 recipients again.

Should the code 452 be treated as an error code, or RemoteDelivery remove the validSent from the list before 'continu'ing to the next server ?