BINARY vs BODY and "invalid" messages

[tomsommer]

Roundcube uses BINARY.PEEK whenever possible, however this prevents the display of broken mails, that usually work with BODY.PEEK

BINARY.PEEK returns UID NO [PARSE] Invalid data in MIME part (0.012 + 0.000 + 0.011 secs) whereas BODY.PEEK returns the mail body.

Perhaps add a fallback to BODY.PEEK if BINARY.PEEK fails?

[alecpl]

There's a fallback, but we use it only on [UNKNOWN-CTE] response. Maybe we should do this also for [PARSE]. What IMAP server/version?

[tomsommer]

Dovecot 2.3.2

[alecpl]

Looks like dovecot 2.3 uses [PARSE] instead of [UNKNOWNCTE]. https://www.dovecot.org/list/dovecot-news/2017-December/000366.html

[tomsommer]

Good catch, Dovecot can return both UNKNOWN-CTE and PARSE now: dovecot/core@ff17313

[alecpl]

Fixed.