pop3 filters that move new message to imap folders mostly won't work

RESOLVED FIXED in Thunderbird 15.0

Status

MailNews Core
Filters
RESOLVED FIXED
5 years ago
4 years ago

People

(Reporter: Bienvenu, Assigned: Bienvenu)

Tracking

({regression})

Trunk
Thunderbird 15.0
x86_64
Windows 7
regression

Thunderbird Tracking Flags

(thunderbird12 fixed, thunderbird13 fixed, thunderbird14 fixed)

Details

(Whiteboard: [fixed in 12.0.1])

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
pop3 incoming mail filters that move messages to imap folders mostly won't work. This is a regression from the pluggable store work. Prior to pluggable stores, we used a single new mail parser for all the messages in a pop3 download session. The moves to imap folders were coalesced by a imap move coalescer owned by the parser. Now, we use one new mail parser per message, which means the imap move coalescing doesn't work, except if the filter happens to apply to the last message downloaded in a download session.
(Assignee)

Comment 1

5 years ago
The relnote would be

pop3 filters that move/copy incoming messages to imap folders will not work, for the most part, except for the last message downloaded in get new mail operation. For this reason, it is probably best to disable such filters until this bug is fixed. Filters that are run manually should work.
Keywords: relnote
(Assignee)

Comment 2

5 years ago
sadly, there's no way to do an automated test for this, so we'd have to fallback onto a litmus test.
Flags: in-litmus?
should this not block bug 402392, rather than be blocked by bug 402392?
Keywords: regression
(Assignee)

Comment 4

5 years ago
(In reply to Wayne Mery (:wsmwk) from comment #3)
> should this not block bug 402392, rather than be blocked by bug 402392?

yeah, sorry, forgot to switch that.
Blocks: 402392
No longer depends on: 402392
(Assignee)

Comment 5

5 years ago
Created attachment 617740 [details] [diff] [review]
proposed fix

This should fix it...
Flags: in-litmus? → in-litmus?(ludovic)
(Assignee)

Comment 6

5 years ago
Comment on attachment 617740 [details] [diff] [review]
proposed fix

this makes pop3 to imap filters work for me, and xpcshell tests all still pass.
Attachment #617740 - Flags: review?(mbanner)

Updated

5 years ago
Status: NEW → ASSIGNED
Attachment #617740 - Flags: review?(mbanner) → review+
Comment on attachment 617740 [details] [diff] [review]
proposed fix

[Triage Comment]
a=me for landing regression fix on branches.
Attachment #617740 - Flags: approval-comm-release+
Attachment #617740 - Flags: approval-comm-beta+
Attachment #617740 - Flags: approval-comm-aurora+
Checked in:
https://hg.mozilla.org/comm-central/rev/2b4524d258f3
https://hg.mozilla.org/releases/comm-aurora/rev/2eef53f31fcf
https://hg.mozilla.org/releases/comm-beta/rev/9d82860ddf8d
https://hg.mozilla.org/releases/comm-release/rev/53fd7df65dba
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
status-thunderbird12: --- → fixed
status-thunderbird13: --- → fixed
status-thunderbird14: --- → fixed
Resolution: --- → FIXED
Whiteboard: [fixed in 12.0.1]
Target Milestone: --- → Thunderbird 15.0
We relnoted this for 12.0 and its fixed in 12.0.1 & later so no need for relnote anymore.
Keywords: relnote
Flags: in-litmus?(ludovic)
You need to log in before you can comment on or make changes to this bug.