From: Eric Wong <e@80x24.org>
Date: Sun, 11 Jan 2015 08:54:25 +0000 (+0000)
Subject: filter: handle missing Content-Type
X-Git-Tag: v1.0.0~1091
X-Git-Url: http://www.git.stargrave.org/?a=commitdiff_plain;h=a33da0b97184a97a01d1f383bdb3faa24a7e648e;p=public-inbox.git

filter: handle missing Content-Type

Some mailers may omit the Content-Type header entirely,
so do detection and try to get the message through.
---

diff --git a/lib/PublicInbox/Filter.pm b/lib/PublicInbox/Filter.pm
index 929a8ffc..6862bb68 100644
--- a/lib/PublicInbox/Filter.pm
+++ b/lib/PublicInbox/Filter.pm
@@ -129,7 +129,7 @@ sub strip_multipart {
 			return;
 		}
 
-		my $part_type = $part->content_type;
+		my $part_type = $part->content_type || '';
 		if ($part_type =~ m!\btext/plain\b!i) {
 			push @keep, $part;
 		} elsif ($part_type =~ $MIME_HTML) {
@@ -140,7 +140,8 @@ sub strip_multipart {
 			# help with.
 
 			push @keep, $part;
-		} elsif ($part_type =~ m!\Aapplication/octet-stream\z!i) {
+		} elsif ($part_type eq '' ||
+		         $part_type =~ m!\Aapplication/octet-stream\z!i) {
 			# unfortunately, some mailers don't set correct types,
 			# let messages of unknown type through but do not
 			# change the sender-specified type