2065695 - nsImapService doesn't perform msgKey<->UID mapping

Status: ASSIGNED

(MailNews Core :: Networking: IMAP, defect)

Description

[Ben Campbell]

The message urls used by nsIMsgMessageService contain message keys.
But nsImapService (which implements nsIMsgMessageService) assumes that those message keys can be used directly as UIDs.
This works fine for now because message keys are UIDs for IMAP, but that'll change with Bug 1806770.

That means that things like nsIMsgMessageService.streamMessage() will stop working for IMAP.

To prepare for that, nsImapService should be changed in advance to use the msgKey<->UID mapping functions declared in nsImapUtils.h.
At the moment those functions just do a 1:1 mapping (message keys passed in are returned verbatim as UIDs and vice versa), but Bug 1806770 will change them to perform actual database lookups.

Comment 1

[Ben Campbell]

Attachment: Bug 2065695 - Fix msgKey<->UID mapping in nsImapService::StreamMessage(). r=#thunderbird-back-end-reviewers

Comment 2

[Ben Campbell]

Attachment: Bug 2065695 - Implement assorted missing UID<->msgKey mapping in nsImapService. r=#thunderbird-back-end-reviewers

Comment 3

[Ben Campbell]

Attachment: WIP: Bug 2065695 - rewrite nsImapService::OfflineAppendFromFile().

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/84f3f219ac3c
Fix msgKey<->UID mapping in nsImapService::StreamMessage(). r=mkmelin