Bug #112063 for Mail-Builder: Fails with Encode 2.79 and later

Mon Feb 15 16:41:37 2016 ANDK [...] cpan.org - Ticket created

Subject:

Fails with Encode 2.79 and later

As per subject. Sample fail reports: 2.79: http://www.cpantesters.org/cpan/report/64716259 2.80: http://www.cpantesters.org/cpan/report/64884407 2.82: http://www.cpantesters.org/cpan/report/65319918 arbitrary more sample fail report by Encode version: http://analysis.cpantesters.org/reports_by_field?distv=Mail-Builder-2.09;field=mod%3AEncode HTH&&Thanks,

Tue Feb 16 02:05:36 2016 SREZIC [...] cpan.org - Correspondence added

On 2016-02-15 16:41:37, ANDK wrote: Show quoted text

> As per subject. Sample fail reports: > > 2.79: http://www.cpantesters.org/cpan/report/64716259 > 2.80: http://www.cpantesters.org/cpan/report/64884407 > 2.82: http://www.cpantesters.org/cpan/report/65319918 > > arbitrary more sample fail report by Encode version: > > http://analysis.cpantesters.org/reports_by_field?distv=Mail-Builder- > 2.09;field=mod%3AEncode > > HTH&&Thanks,

Probably a problem in Encode. See https://rt.cpan.org/Ticket/Display.html?id=111853

Tue Feb 16 02:05:36 2016 The RT System itself - Status changed from 'new' to 'open'

Tue Feb 16 02:06:10 2016 SREZIC [...] cpan.org - Reference to ticket #111853 added

Fri Apr 01 15:25:12 2016 pali [...] cpan.org - Cc PALI added

Fri Apr 01 15:29:29 2016 pali [...] cpan.org - Correspondence added

On Pon Feb 15 16:41:37 2016, ANDK wrote: Show quoted text

> As per subject. Sample fail reports: > > 2.79: http://www.cpantesters.org/cpan/report/64716259 > 2.80: http://www.cpantesters.org/cpan/report/64884407 > 2.82: http://www.cpantesters.org/cpan/report/65319918 > > arbitrary more sample fail report by Encode version: > > http://analysis.cpantesters.org/reports_by_field?distv=Mail-Builder- > 2.09;field=mod%3AEncode > > HTH&&Thanks,

Probably you are incorrectly using MIME encoder and are trying to encode something which should not be encoded. E.g. MIME-encoded phrase part of email addresses must not be quoted. Either do not MIME encode it or do not quote it. Anyway, in Encode 2.83 I finally fixed MIME encoder/decoder, so if you still have some problems, I can try to help.

Fri Apr 01 19:01:16 2016 ANDK [...] cpan.org - Correspondence added

On 20160401T192929Z, PALI wrote: Show quoted text

> [...] in Encode 2.83 I finally fixed MIME > encoder/decoder, so if you still have some problems, I can try to > help.

For what it is wirth, http://analysis.cpantesters.org/solved?distv=Mail-Builder-2.09#mod%3AEncode shows that Encode 2.83 still leads to failing tests in MIME::Builder 2.09.

Sat Apr 02 03:28:07 2016 MAROS [...] cpan.org - Correspondence added

Thank you for your bug report. This issue was fixed in 2.10 and encoding will only happen if non-ascii characters are present

Sat Apr 02 03:28:07 2016 MAROS [...] cpan.org - Status changed from 'open' to 'resolved'

Sat Apr 02 05:50:55 2016 pali [...] cpan.org - Correspondence added

On Sob Apr 02 03:28:07 2016, MAROS wrote: Show quoted text

> Thank you for your bug report. > > This issue was fixed in 2.10 and encoding will only happen if non- > ascii characters are present

I looked into Mail/Builder/Address.pm code and you still have it incorrect. There is line: sprintf '"%s" <%s>',Mail::Builder::Utils::encode_mime($name),$self->email; You can either MIME encode or quote phrase part. Not both! But you quote %s always, which is wrong. Also you have incorrect check in Mail/Builder/Utils.pm code. If somebody wants to set phrase name to something like this "=?a?b?c?=" then you need to MIME encode it too (because string has form for MIME word). Also same problem happens if phrase name contains " or ', then you need escape it.

Sat Apr 02 05:51:31 2016 pali [...] cpan.org - Status changed from 'resolved' to 'open'

Mon Mar 13 15:19:58 2017 MAROS [...] cpan.org - Correspondence added

Thank you for your bug report. This issue should be fixed in 2.12

Mon Mar 13 15:19:59 2017 MAROS [...] cpan.org - Status changed from 'open' to 'resolved'