Thread is not reopened because of misinterpreted header

pronego-mls

Hi,
An existing thread is not reopened when a new customer email is parsed.
Debugging revealed that the responsible part is the function isBounce() from class.filter.php.
Here, the line 'Return-Path' => array('strcmp', array('<>'), 0), is responsible for it.
The incoming emails very often have "Return-path: <>" set which often happens when emails are sent from webmailers. Does it make sense to just comment this condition out? Can it be removed in general?
Thank you for some suggestions.

KevinTheJedi

pronego-mls

RFC docs says (reference):

While the syntax indicates that a route specification is optional, every attempt should be made to provide that information in this field.

So while it’s not required it’s standard across most major providers and should be added appropriately. We see it as an error/bounce as if it does bounce/fail or something we and the server have no idea who to return it to.

You can remove it if you want but this will not be removed in core. We’ve been using this for years without issues.

Cheers.

ntozier

Please help us to help you by reading and following the posting guidelines located in this thread: Please read before requesting assistance. The more information you give us the better we will be able to assist you. Thank you.

Environment details? (see Admin panel -> Dashboard -> Information)

pronego-mls

KevinTheJedi

Thank you for the quick response, I agree to your explanation.
However, I'm still wondering because I did the testing with GMX, which is a big (official) email provider in Germany and it shouldn't happen then.

But this is not relevant to osticket then.