On Mon, Aug 10, 2015 at 10:43:39PM +0300, Max Kosmach wrote:
> When SMIME support in 4.2.8-3 is enabled I have a problem with non-ascii
> signed emails.
> When this email received by RT I see error like this in logs:
>
> Aug 10 17:01:59 rt postfix/local[4066]: D66DD14A005:
> to=<rt@localdomain>, relay=local, delay=158, delays=158/0.03/0/0.4,
> dsn=4.3.0, status=deferred (temporary failure. Command output: RT
> server error. The RT server which handled your email did not behave
> as expected. It said: Cannot decode string with wide characters at
> /usr/lib/i386-linux-gnu/perl/5.20/Encode.pm line 175. Stack:
> [/usr/lib/i386-linux-gnu/perl/5.20/Encode.pm:175]
> [/usr/share/request-tracker4/lib/RT/Interface/Email/Auth/Crypt.pm:177]
> [/usr/share/request-tracker4/lib/RT/Interface/Email.pm:1300]
> [/usr/share/request-tracker4/html/REST/1.0/NoAuth/mail-gateway:61]
> )
>
> Same problem with 4.2.11-1 when I send same email from CLI:
>
> Cannot decode string with wide characters at
> /usr/lib/i386-linux-gnu/perl5/5.20/Encode.pm line 200.
>
> Stack:
> [/usr/lib/i386-linux-gnu/perl5/5.20/Encode.pm:200]
> [/usr/share/request-tracker4/lib/RT/Interface/Email/Auth/Crypt.pm:177]
> [/usr/share/request-tracker4/lib/RT/Interface/Email.pm:1321]
> [/usr/share/request-tracker4/html/REST/1.0/NoAuth/mail-gateway:61]
>
> RT server error.
>
> I think patch below fixes this problem, but I don't know is this patch fully
> correct or not:
>
> ===
> diff -urN /usr/share/request-tracker4/lib/RT/Crypt.pm
> /usr/share/request-tracker4/lib/RT/Crypt.pm.new
> --- /usr/share/request-tracker4/lib/RT/Crypt.pm 2015-05-10 22:25:52.000000000
> +0300
> +++ /usr/share/request-tracker4/lib/RT/Crypt.pm.new 2015-08-10
> 22:14:50.916118963 +0300
> @@ -545,7 +545,7 @@
> my $modify = $res{status_on}->head->modify;
> $res{status_on}->head->modify(1);
> $res{status_on}->head->add(
> - "X-RT-" . $protected->{'Protocol'} . "-Status" => $res{'status'}
> + "X-RT-" . $protected->{'Protocol'} . "-Status" =>
> Encode::encode( "UTF-8", $res{'status'})
> );
> $res{status_on}->head->modify($modify);
>
>
> ===
>
> If this patch correct - please apply it to debian package.
Hello,
As I believe I said to you in private mail, but for the bug record:
I'm not able to verify that this is correct - I don't use SMIME - so
I would like to hear feedback from RT upstream about the patch before
applying it.
Thanks,
Dominic.
