Don't split Quoted Printable characters during folding procedure

Author: Gonzih

header.go

@@ -3,6 +3,7 @@ package message
 import (
 	"mime"
 	"net/textproto"
+	"regexp"
 	"strings"
 
 	"github.com/emersion/go-message/charset"
@@ -40,6 +41,8 @@ func formatHeaderField(k, v string) string {
 		return s + "\r\n"
 	}
 
+	qpReg := regexp.MustCompile("(=[0-9A-Z]{2,2})+")
+
 	first := true
 	for len(v) > 0 {
 		maxlen := maxHeaderLen
@@ -60,7 +63,13 @@ func formatHeaderField(k, v string) string {
 			}
 		} else {
 			// Find the closest whitespace before i
-			foldAt = strings.LastIndexAny(v[:foldBefore], " \t\n")
+			foldAtQP := qpReg.FindAllStringIndex(v[:foldBefore], -1)
+			foldAtEOL := strings.LastIndexAny(v[:foldBefore], " \t\n")
+			if len(foldAtQP) > 0 {
+				foldAt := foldAtQP[len(foldAtQP)-1][0]
+			} else {
+				foldAt = foldAtEOL
+			}
 			if foldAt == 0 {
 				// The whitespace we found was the previous folding WSP
 				foldAt = foldBefore - 1

header_test.go

@@ -59,6 +59,11 @@ var formatHeaderFieldTests = []struct {
 		v:         "This is        yet          \t  another    subject          \t                   with many         whitespace      characters",
 		formatted: "Subject: This is        yet          \t  another    subject          \t       \r\n            with many         whitespace      characters\r\n",
 	},
+	{
+		k:         "Subject",
+		v:         "=?utf-8?q?=E2=80=9CDeveloper_reads_customer_requested_change.=E2=80=9D=0A?= =?utf-8?q?=0ACaravaggio=0A=0AOil_on...?=",
+		formatted: "Subject: =?utf-8?q?=E2=80=9CDeveloper_reads_customer_requested_change.\r\n =E2=80=9D=0A?= =?utf-8?q?=0ACaravaggio=0A=0AOil_on...?=\r\n",
+	},
 	{
 		k:         "DKIM-Signature",
 		v:         "v=1;\r\n h=From:To:Reply-To:Subject:Message-ID:References:In-Reply-To:MIME-Version;\r\n d=example.org\r\n",