1665287 - Wrong encoding of decrypted messages from Symantec Encryption Desktop

Status: RESOLVED FIXED

(MailNews Core :: Security: OpenPGP, defect)

Description

[Marcel Dopita]

Attachment: Enigmail vs native

User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:80.0) Gecko/20100101 Firefox/80.0

Steps to reproduce:

I sent an encrypted email to an user of Symantec Encryption Desktop who replied.

Actual results:

Reply from Symantec Encryption Desktop is displayed in wrong encoding.

Expected results:

The same email is displayed correctly in Thunderbird 68.x with Enigmail.

Comment 1

[Marcel Dopita]

I have just sent an sample message to [e-mail].

Comment 2

[Kai Engert (:KaiE:)]

Marcel, thanks for the report and the test message.

I understand that in your screenshot, the first line is the "expected" display, and the second line shows the "broken" display, right?

Using your test message, I see the second (broken) line shown.
If I back out locally, I see the first line from your screenshot.

This is a regression from bug 1659443.

I suggest that we back out bug 1659443 and try to find a better fix.

Comment 3

[Kai Engert (:KaiE:)]

Attachment: Bug 1665287 - Backout initial fix for bug 1659443 because it regressed international characters. r=mkmelin

Comment 4

[Marcel Dopita]

Yes, in the screenshot, the first line is expected display and second one is actual (broken) display.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/66f6e3dca6d8
Backout initial fix for bug 1659443 because it regressed international characters. r=mkmelin DONTBUILD

Comment 6

[Kai Engert (:KaiE:)]

Comment on attachment 9178312 [details]
Bug 1665287 - Backout initial fix for bug 1659443 because it regressed international characters. r=mkmelin

[Approval Request Comment]
Regression caused by (bug #): 1659443
User impact if declined: International users cannot read decrypted messages in their language
Testing completed (on c-c, etc.): Backout, so code is known to have worked previously.
Risk to taking this patch (and alternatives if risky): minimal, but restores slowness

Comment 7

[Wayne Mery (:wsmwk)]

Comment on attachment 9178312 [details]
Bug 1665287 - Backout initial fix for bug 1659443 because it regressed international characters. r=mkmelin

[Triage Comment]
Approved for beta
Approved for esr78

Comment 8

[Kai Engert (:KaiE:)]

82.0b2
https://hg.mozilla.org/releases/comm-beta/rev/946da3756625c0931c8230ff71c333f634ff9b01

Comment 9

[Kai Engert (:KaiE:)]

78.3.2
https://hg.mozilla.org/releases/comm-esr78/rev/0730c6a1c8df571777613dec88854feae355ea9e