Created attachment 410861 [details] [diff] [review]
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 on attachment 410861 [details] [diff] [review]
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.
fixed on trunk and branch, thx!
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...