XF\EmailBounce\Parser::getBounceTypeFromStatus does not handle statuses with multiple digits

[DragonByte Tech]

Affected version

2.3.7

Amazon SES will sometimes send a status such as 5.1.10, which will explicitly fail on this regex:

if (!$statusCode || !preg_match('#^\d\.\d\.\d$#', $statusCode))

Suggested fix:

if (!$statusCode || !preg_match('#^\d+\.\d+\.\d+$#', $statusCode))

 

[Alpha1]

Especially .edu email services have a tendency to use double digits, which is why bounce processing for .edu addresses often fails.

 

[Alpha1]

Here's an example (AWS SNS Topic) of a double digit status code often used by .edu providers:

"bounceType\":\"Transient\",\"bounceSubType\":\"General\",\"bouncedRecipients\":[{\"emailAddress\":\"xxx@iup.edu\",\"action\":\"failed\",\"status\":\"5.4.14\",\"diagnosticCode\":\"smtp;554 5.4.14 Hop count exceeded - possible mail loop ATTR34

.edu email has a high bounce rate due to users moving on after their study or employment. Especially on forums with an older userbase.
Often the .edu email is forwarded to another inbox, if that no longer accepts email it will send it back. Forwarding chains or loops are common in this subset of users.
Microsoft Exchange Server uses these double digit status codes and MES is often used by .edu providers.