changeset 20890:d946c76fb3d4

lib-imap: imap_bodystructure_parse() ignores text/plain flag mismatch for now. Maybe we can enable it some day far into the future, but for now this just causes unnecessary cache file rebuilds.
author Timo Sirainen <timo.sirainen@dovecot.fi>
date Mon, 10 Oct 2016 20:24:41 +0300
parents f8ba998a6b7c
children d2872ec409ec
files src/lib-imap/imap-bodystructure.c
diffstat 1 files changed, 6 insertions(+), 0 deletions(-) [+]
line wrap: on
line diff
--- a/src/lib-imap/imap-bodystructure.c	Mon Oct 10 20:23:47 2016 +0300
+++ b/src/lib-imap/imap-bodystructure.c	Mon Oct 10 20:24:41 2016 +0300
@@ -740,10 +740,16 @@
 	text = strcasecmp(content_type, "text") == 0;
 	message_rfc822 = strcasecmp(content_type, "message") == 0 &&
 		strcasecmp(subtype, "rfc822") == 0;
+#if 0
+	/* Disabled for now. Earlier Dovecot versions handled broken
+	   Content-Type headers by writing them as "text" "plain" to
+	   BODYSTRUCTURE reply, but the message_part didn't have
+	   MESSAGE_PART_FLAG_TEXT. */
 	if (text != ((part->flags & MESSAGE_PART_FLAG_TEXT) != 0)) {
 		*error_r = "message_part text flag doesn't match BODYSTRUCTURE";
 		return -1;
 	}
+#endif
 	if (message_rfc822 != ((part->flags & MESSAGE_PART_FLAG_MESSAGE_RFC822) != 0)) {
 		*error_r = "message_part message/rfc822 flag doesn't match BODYSTRUCTURE";
 		return -1;