]> Sergey Matveev's repositories - public-inbox.git/commitdiff
msgiter: attempt to decode all text/* bodies
authorEric Wong <e@80x24.org>
Thu, 31 Oct 2019 03:12:19 +0000 (03:12 +0000)
committerEric Wong <e@80x24.org>
Thu, 31 Oct 2019 03:12:37 +0000 (03:12 +0000)
We want to index text/x-patch and text/x-diff, at least,
since "git format-patch" can generate a patch series as
attachments using --attach.

lib/PublicInbox/MsgIter.pm

index f11ba223c93362a985285555e4113feab87684f1..d9df32abc37c7fc7da68c0e09ed2f942ddbb4827 100644 (file)
@@ -45,12 +45,18 @@ sub msg_part_text ($$) {
        # times when it should not have been:
        #   <87llgalspt.fsf@free.fr>
        #   <200308111450.h7BEoOu20077@mail.osdl.org>
-       if ($err && ($ct =~ m!\btext/plain\b!i ||
+       if ($err && ($ct =~ m!\btext/\b!i ||
                        $ct =~ m!\bmultipart/mixed\b!i)) {
-               # Try to assume UTF-8 because Alpine seems to
-               # do wacky things and set charset=X-UNKNOWN
-               $part->charset_set('UTF-8');
-               $s = eval { $part->body_str };
+               my $cte = $part->header_raw('Content-Transfer-Encoding');
+               if (defined($cte) && $cte =~ /\b7bit\b/i) {
+                       $s = $part->body;
+                       $err = undef if $s =~ /\A[[:ascii:]]+\z/s;
+               } else {
+                       # Try to assume UTF-8 because Alpine seems to
+                       # do wacky things and set charset=X-UNKNOWN
+                       $part->charset_set('UTF-8');
+                       $s = eval { $part->body_str };
+               }
 
                # If forcing charset=UTF-8 failed,
                # caller will warn further down...