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

RESOLVED FIXED in Thunderbird 15.0

Status

defect
RESOLVED FIXED
7 years ago
6 years ago

People

(Reporter: Bienvenu, Assigned: Bienvenu)

Tracking

({regression})

Trunk
Thunderbird 15.0
x86_64
Windows 7
Dependency tree / graph

Thunderbird Tracking Flags

(thunderbird12 fixed, thunderbird13 fixed, thunderbird14 fixed)

Details

(Whiteboard: [fixed in 12.0.1])

Attachments

(1 attachment)

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.
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
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
(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
Posted patch proposed fixSplinter Review
This should fix it...
Flags: in-litmus? → in-litmus?(ludovic)
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)
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+
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.