Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support mail box names containing square brackets #552

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

jnschulze
Copy link

@jnschulze jnschulze commented Apr 30, 2016

The "Airmail" app for OS X creates a mailbox called "[Airmail]" and some sub-folders. This doesn't seem to be supported, as the result passed to the "getBoxes()" callback looks broken. It's expected to return something like:

[Airmail], [Airmail].To Do, [Airmail].Done, [Airmail].Memo, [Airmail].Snooze
however, it returns
[Airmail], Airmail, [Airmail].To Do (all other boxes are missing)

@mscdex
Copy link
Owner

mscdex commented Apr 30, 2016

Can you please include a test as well?

@jnschulze
Copy link
Author

Sure, see https://github.com/nischu7/node-imap/blob/master/test/test-parse-mailbox-names.js
Vary line 39 by removing the explicit "false" ;)

Niklas

@mscdex
Copy link
Owner

mscdex commented Apr 30, 2016

Sorry, I meant adding an export for parseBoxList() and testing that, so that the new test fails before the changes and succeeds after the changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants