Split body of mails not respecting RFC2822

[bapt]

For mails where the body does not respect RFC2822, try to split by words finding the last space before 1000's character

If no spaces are found then consider the mail to be malformed anyway

This should address #18

[corecode]

If we split some lines, I think we should split all lines. For simplicity's sake, I'd split them strictly at byte 990 or so...

[bapt]

So you want me to remove the discovery of the space and always split long lines?

[bapt]

done

[bapt]

I think I addressed all the comments

[emaste]

Ping?

[kev009]

We ran into unexpected issues that this would fix when trying to switch to dma. For instance, cron jobs would fail if there was a long line on stdout/stderr as it gets piped to sendmail.

[jailbird777]

Just wanted to comment that while this PR seems to still apply cleanly, when we deployed it we instantly had complaints about some automated emails getting rejected with a corrupted queue error, so failing this check:

https://github.com/corecode/dma/blob/master/net.c#L582

I sadly never got a copy of one in order to replicate the issue on-demand, so we just reverted the patch for now. If I find a way to replicate the issue, I'll dig some more into this.

[mslehto]

Hi @jailbird777

Regarding your comment on on 20 Feb 2020, can you try to replicate your issue with instructions on FreeBSD PR266629 ?

And, if running FreeBSD, you can also try corresponding patches.