[hobleyd]

Problem

StatusParser required the mailbox name to be double-quoted: RegExp(r'(STATUS "[^"]+?" )(.*)'). RFC 3501 allows the mailbox to be sent as an atom, and Dovecot does so (* STATUS INBOX (MESSAGES 231 UNSEEN 5 ...)), so the regex never matched and messagesExists/messagesUnseen/uidNext were silently left at 0 for every mailbox on such a server.

Fix

The regex accepts either a quoted string or an atom: (STATUS (?:"[^"]+?"|[^ (]+) )(.*). The existing test for a quoted name containing brackets still passes.

Tests

New test in status_parser_test.dart for STATUS INBOX (MESSAGES 231 UNSEEN 5 UIDNEXT 4392). Fails against main.

[robert-virkus]

<3

[christhomas]

Hi, just wanted to ping you here because this fix has no released version, can we do that?

[hobleyd]

Hi, just wanted to ping you here because this fix has no released version, can we do that?

I use a dependency override in the pubspec.yaml while waiting for the release. Check it out locally and use:

dependency_overrides:
  # Locally patched copy of enough_mail 2.1.7. Every patch has been raised
  # against Enough-Software/enough_mail; drop this override once all of these
  # have merged and shipped in a release:
  #   #287 quoted-printable attachments decoded to bytes
  #   #288 pending IMAP commands fail when the connection is lost
  #   #289 mailbox names containing parentheses (LIST parser + quoting)
  #   #290 STATUS with an unquoted mailbox name (Dovecot)
  #   #291 STATUS records HIGHESTMODSEQ
  #   #292 APPEND internalDate + appendMessageBytes (stacked on #289)
  #   #293 bare-LF line endings in a fetched BODY[] literal
  #   #294 SMTP DATA kept the folded continuation lines of a long Bcc header
  enough_mail:
    path: packages/enough_mail