1565887 - Fix IMAP's and NNTP's {Get|Set}OriginalURI()

Status: NEW

(MailNews Core :: General, defect)

Description

[Jorg K (CEST = GMT+2)]

Attachment: 1564094-OriginalURI.patch

+++ This bug was initially created as a clone of Bug #1564094 +++

In bug 1564094 we detected that IMAP's and NNTP's {Get|Set}OriginalURI() don't appear to be right.

That caused trouble for "view source " since the set call received a view-source:imap URI, but the get call returned an imap: URI. See 1564094 comment #19 for debugging output, there's also a debug patch in that bug for get that debugging output.

Something like this would be correct:
https://searchfox.org/comm-central/rev/e9439ba2501591b8d9890bd6973d74bd3c06da3f/mailnews/base/util/nsMsgProtocol.cpp#505-512

I'll attach a patch which fixes IMAP, but Boris thinks that this is very risky, since it could regress bug 193317 or bug 1312314.

Further reading:
Bug 1564094 comment #11, 2nd dot point; bug 1564094 comment #12
Bug 1312314 comment #32

Comment 1

[Wayne Mery (:wsmwk)]

So this is a version 70 regression?

Comment 2

[Jorg K (CEST = GMT+2)]

No regression, has always been wrong.