The default bug view has changed. See this FAQ.

handle non '/' hierarchy delimiters during first time xlist special folder discovery

RESOLVED FIXED in Thunderbird 3.0rc1

Status

MailNews Core
Networking: IMAP
RESOLVED FIXED
8 years ago
7 years ago

People

(Reporter: Bienvenu, Assigned: Bienvenu)

Tracking

({fixed-seamonkey2.0.1})

Trunk
Thunderbird 3.0rc1
x86
Windows NT
fixed-seamonkey2.0.1

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

8 years ago
Created attachment 410861 [details] [diff] [review]
proposed fix

If an imap server supports xlist but doesn't use '/' as the hierarchy delimiter (e.g., Cyrus, which is used by fastmail), on first connection, our second xlist command doesn't use the correct hierarchy delimiter, so special folders aren't recognized until a restart, which isn't a nice user experience.

This patch is very simple, and only affects imap servers that support xlist (gmail and dovecot currently, cyrus/fastmail soon).

I'd love to have this make 3.0 so we work well with fastmail.
Attachment #410861 - Flags: superreview?(dmose)
Attachment #410861 - Flags: review?(dmose)

Updated

8 years ago
Attachment #410861 - Attachment is patch: true
Attachment #410861 - Attachment mime type: application/octet-stream → text/plain

Updated

8 years ago
Attachment #410861 - Flags: superreview?(dmose)
Attachment #410861 - Flags: superreview+
Attachment #410861 - Flags: review?(dmose)
Attachment #410861 - Flags: review+
Attachment #410861 - Flags: approval-thunderbird3+
Comment on attachment 410861 [details] [diff] [review]
proposed fix

r+sr+a=dmose.  If it makes sense, add an XXX comment that when we're less risk-averse we could make that setter call not be conditional on Inbox, so that m_runningUrl would reflect reality better and perhaps prevent future bugs.
(Assignee)

Comment 2

8 years ago
fixed on trunk and branch, thx!
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 3.0rc1

Comment 3

8 years ago
Confirmed works with my testing XLIST branch at FastMail - will push this back to Cyrus, so it should be useful for other Cyrus sites before long too.

I suspect there will be some debate about hard coded config file entries versus cyradm settable annotations before it goes mainstream at the server end though...

Bron.

Updated

7 years ago
Keywords: fixed-seamonkey2.0.1
You need to log in before you can comment on or make changes to this bug.