TYPO3 Core

Hi Stephan,

thank you for your patch!

Please send it to the Core List!
To see what to do, check out http://typo3.org/teams/core/core-mailinglist-rules/

Yes, the issue still exists. The else-if statement to set the correct reason (bad host instead of unknown) for return of non-qmail, non-postfix return mails has to be added in line 199 in 4.5.26.

I guess this has not changed fundamentally in 6.1, so it should apply there too.

Since the PHP Manual states not to use preg_match for pure checking if one string contains another for performance reasons I modified the code (as well for the added else-if for bad host as for the following two cases:

} else if (stristr($cp['reason_text'], 'Host or domain name not found') || stristr($cp['reason_text'], 'Host not found')) {
    $cp['reason'] = 2; // Bad host
} else if (stristr($cp['reason_text'], 'over quota') || stristr($cp['reason_text'], 'mailbox full')) {
    $cp['reason'] = 551; // Mailbox full
} else if (stristr($cp['reason_text'], 'Error in Header')) {
    $cp['reason'] = 554;