Skip to content

Fixing regex for identifying IMAP folders by supporting "." delimiter#11

Open
Mocuto wants to merge 3 commits intoMiksus:masterfrom
Mocuto:master
Open

Fixing regex for identifying IMAP folders by supporting "." delimiter#11
Mocuto wants to merge 3 commits intoMiksus:masterfrom
Mocuto:master

Conversation

@Mocuto
Copy link
Copy Markdown

@Mocuto Mocuto commented Oct 18, 2023

In the series of commits, I update the EmailBox class such that it supports delimiting the inbox and flags via a ".", which is the format used by some IMAP servers. I test these changes locally and observe that they work correctly.

In the `update` method in the EmailBox class, there is a regex for checking the returned mail folders from the IMAP_SSL connection. However, this regex does not catch the case where in an inbox is specified in the format `(\\HasNoChildren) "." INBOX`. This is the format returned by my IMAP server and is valid.

This commit modifies the regex to allow either the '/' or '.' to separate the flags from the folder name.
Fixing regex for identifying IMAP inbox folders
@ddorn
Copy link
Copy Markdown

ddorn commented Jan 20, 2024

+1
I tested this modification and it also makes it work with my IMAP server that seems to use a "." instead of "/"

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