FixSubstringBoundaries errors

[Shawn Iverson]

Removed FixSubstringBoundaries for next release.  The problem has long
since been resolved and this workaround is no longer necessary.  I left the
code commented for future reference.

On Wed, Dec 5, 2018 at 11:53 AM Shawn Iverson <iversons at rushville.k12.in.us>
wrote:

> I'll take a deep dive hopefully tonight or tomorrow.  I'm not sure that
> function is even relevant any more...
>
> On Wed, Dec 5, 2018 at 11:33 AM Radoslav Bodó <bodik at cesnet.cz> wrote:
>
>> Hello,
>>
>> after the latest upgrade of our server from MailScanner 4 to 5.1.2 we
>> have identified that FixSubstringBoundaries is not working properly.
>>
>>
>>
>> Currently:
>>
>> * the function FixSubstringBoundaries(...) updates the body and headers
>> in the processed $message structure, but final delivered message
>> contains original non-updated Conten-Type boundary header
>>
>> * that renders updated messages unusable on the server/client side. In
>> some cases the messages are just not properly displayed (roundcube), in
>> some other cases might be deleted silently on the background by the
>> client (thunderbird).
>>
>> * possibly corrupted messages can be identified from the logs by
>> 'Content Checks: Fixed awkward MIME boundary for Cyrus IMAP server'
>>
>>
>>
>> System affected:
>>
>>         debian 9.5 stretch
>>         postfix 3.1.8-0+deb9u1
>>         mailscanner 5.1.2-2
>>         perl 5.24.1-3+deb9u4
>>         libmime-tools-perl 5.508-1
>>
>>
>>
>> The attachments of this message contains data for reproducing the issue:
>>
>> * mailboundary1.txt            .. test message to reproduce
>> * mailboundary1-corrupted.txt  .. resulting message documenting the fail
>> * sendmail.py                  .. testing helper
>>
>> cat /tmp/mailboundary1.txt | /tmp/sendmail.py --sender
>> s at webmail-test.redacted --to s at webmail-test.redacted --raw
>>
>>
>>
>>
>>
>> The only workaround I've been able to come up is to disable the feature
>> in the code by commenting out
>>
>>
>> https://github.com/MailScanner/v5/blob/master/common/usr/share/MailScanner/perl/MailScanner/SweepContent.pm#L215
>>
>> for the time being.
>>
>>
>>
>>
>> I've tried to track down message processing down to
>>
>>
>> https://github.com/MailScanner/v5/blob/master/common/usr/share/MailScanner/perl/MailScanner/Message.pm#L5348
>>
>> but since my knowledge of mailscanner and postfix internals are poor
>> I've got lost in DeliverModifiedBody(...)
>>
>>
>>
>> Any help or idea would be highly appreciated.
>>
>> Thank you
>> bodik
>>
>>
>> --
>> MailScanner mailing list
>> mailscanner at lists.mailscanner.info
>> http://lists.mailscanner.info/mailman/listinfo/mailscanner
>>
>>
>
> --
> Shawn Iverson, CETL
> Director of Technology
> Rush County Schools
> 765-932-3901 option 7
> iversons at rushville.k12.in.us
>
>
>

-- 
Shawn Iverson, CETL
Director of Technology
Rush County Schools
765-932-3901 option 7
iversons at rushville.k12.in.us
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.mailscanner.info/pipermail/mailscanner/attachments/20181210/8cf53677/attachment.html>