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

Status: RESOLVED FIXED

(MailNews Core :: Networking: IMAP, defect)

Description

[David :Bienvenu]

Attachment: 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.

Comment 1

[Dan Mosedale (:dmosedale, :dmose)]

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.

Comment 2

[David :Bienvenu]

fixed on trunk and branch, thx!

Comment 3

[Bron Gondwana]

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.