Support to process Header only emails.

reinaldocoelho · reinaldocoelho · commit ee75b612b683 · 2017-08-29T11:37:26.000-03:00
diff --git a/.vs/ActiveUp.Net-vs2017/v15/sqlite3/storage.ide b/.vs/ActiveUp.Net-vs2017/v15/sqlite3/storage.ide
diff --git a/Class Library/ActiveUp.Net.Common/MimeBody.cs b/Class Library/ActiveUp.Net.Common/MimeBody.cs
@@ -84,7 +84,7 @@ public string Text
                 else
                     return ProductHelper.GetTrialString(this._text, TrialStringType.LongText);
 #else
-                string ret = _text;
+                string ret = _text ?? "";
                 ret = ret.Replace("FLAGS (Seen)", string.Empty);
                 return ret;
 #endif
diff --git a/Class Library/ActiveUp.Net.Common/Parser.cs b/Class Library/ActiveUp.Net.Common/Parser.cs
@@ -530,13 +530,14 @@ public static MimePart ParseMimePart(byte[] binaryData, Message message)
                 var headerEnd = Regex.Match(part.OriginalContent, @".(?=\r?\n\r?\n)").Index + 1;
                 var bodyStart = Regex.Match(part.OriginalContent, @"(?<=\r?\n\r?\n).").Index;
 
-                //TODO: remove this workaround
-                if (bodyStart == 0)
+                // Solve header only situations.
+                if (bodyStart == 0 && headerEnd == 1)
                 {
-                    var indexBody = part.OriginalContent.IndexOf("\r\n\r\n");
-                    if (indexBody > 0)
-                        bodyStart = indexBody;
+                    ParseHeaderFields(part, part.OriginalContent.Length);
+                    return part;
                 }
+
+                // Process Header and Body
                 if (part.OriginalContent.Length >= headerEnd)
                 {
                     ParseHeaderFields(part, headerEnd);
diff --git a/Class Library/ActiveUp.Net.Tests/ActiveUp.Net.Tests.csproj b/Class Library/ActiveUp.Net.Tests/ActiveUp.Net.Tests.csproj
@@ -133,6 +133,9 @@
     <None Include="resource\multipart_body_text_and_html.eml">
       <CopyToOutputDirectory>Always</CopyToOutputDirectory>
     </None>
+    <None Include="resource\only_header.eml">
+      <CopyToOutputDirectory>Always</CopyToOutputDirectory>
+    </None>
     <None Include="resource\quoted-printable-notepad-linebreak.eml">
       <CopyToOutputDirectory>Always</CopyToOutputDirectory>
     </None>
diff --git a/Class Library/ActiveUp.Net.Tests/Common/ParserTests.cs b/Class Library/ActiveUp.Net.Tests/Common/ParserTests.cs
diff --git a/Class Library/ActiveUp.Net.Tests/resource/only_header.eml b/Class Library/ActiveUp.Net.Tests/resource/only_header.eml
@@ -0,0 +1,39 @@
+Received: (qmail 54705 invoked by uid 89); 28 Aug 2017 14:51:48 -0300
+Delivered-To: postseller@destinatary.com.br
+Received: (qmail 54686 invoked from network); 28 Aug 2017 14:51:47 -0300
+Received: by simscan 1.4.0 ppid: 54607, pid: 54648, t: 2.9204s
+         scanners: clamav: 0.99.2/m:/d:23727 spam: 3.4.1
+X-Spam-Checker-Version: SpamAssassin: -last, FreeBSD Brasil LTDA rulesets: Yes
+X-Spam-Status: No, hits=-1.6 required=5.0
+Received: from unknown ([200.147.97.220])
+          (envelope-sender <sender@host.com.br>)
+          by color.serveru.us (qmail-ldap-1.03) with SMTP
+          for <postseller@destinatary.com.br>; 28 Aug 2017 14:51:44 -0300
+Received: from localhost (localhost.localdomain [127.0.0.1])
+  by a4-salsa1.host.com.br (Postfix) with ESMTP id C56FB380009F
+  for <postseller@destinatary.com.br>; Mon, 28 Aug 2017 14:55:45 -0300 (BRT)
+Received: from a4-salsa1.host.intranet (localhost.localdomain [127.0.0.1])
+  by a4-salsa1.host.com.br (Postfix) with ESMTP id AD4CF380009B
+  for <postseller@destinatary.com.br>; Mon, 28 Aug 2017 14:55:45 -0300 (BRT)
+DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=host.com.br; s=afl;
+  t=1503942945; bh=frcCV1k9oG9oKj3dpUqdJg1PxRT2RSN/XKdLCPjaYaY=;
+  h=Date:From:To:In-Reply-To:References:Subject;
+  b=ikMN+aVF4eIDnS6T6mrnzUNHPBtTqrZJgTuZh3dPK/WqrE5JQzp6izefol5kMNHyi
+   1CLGtQ3UutOWtdD1g2JsugJD0SrwuVGxQ9bkcTZX6zRkkETnPKJj/Y6XBxHrSZhWEd
+   2Zq0nQLoGZOwDDzwWoHHUpnKasVy7TXHp8/NrBO4=
+Received: from localhost (a4-winter6.host.intranet [10.131.133.132])
+  by a4-salsa1.host.intranet (Postfix) with ESMTP id A51303800081
+  for <postseller@destinatary.com.br>; Mon, 28 Aug 2017 14:55:45 -0300 (BRT)
+Date: Mon, 28 Aug 2017 14:55:45 -0300
+From: "sender@host.com.br" <sender@host.com.br>
+To: "postseller@destinatary.com.br" <postseller@destinatary.com.br>
+Message-ID: <59a459219e757_52803fc2847231301918b@a4-winter6.mail>
+In-Reply-To: 
+References: 
+Subject: I'm put all my text message on subject, then no has body in this e-mail.
+ second line exists too.
+Mime-Version: 1.0
+Content-Type: text/html
+Content-Transfer-Encoding: 7bit
+X-SenderIP: 200.164.35.146
+X-SIG5: d41d8cd98f00b204e9800998ecf8427e

Original file line number	Diff line number	Diff line change
`@@ -530,13 +530,14 @@ public static MimePart ParseMimePart(byte[] binaryData, Message message)`
`530`	`530`	`var headerEnd = Regex.Match(part.OriginalContent, @".(?=\r?\n\r?\n)").Index + 1;`
`531`	`531`	`var bodyStart = Regex.Match(part.OriginalContent, @"(?<=\r?\n\r?\n).").Index;`
`532`	`532`
`533`		`- //TODO: remove this workaround`
`534`		`- if (bodyStart == 0)`
	`533`	`+ // Solve header only situations.`
	`534`	`+ if (bodyStart == 0 && headerEnd == 1)`
`535`	`535`	`{`
`536`		`- var indexBody = part.OriginalContent.IndexOf("\r\n\r\n");`
`537`		`- if (indexBody > 0)`
`538`		`- bodyStart = indexBody;`
	`536`	`+ ParseHeaderFields(part, part.OriginalContent.Length);`
	`537`	`+ return part;`
`539`	`538`	`}`
	`539`	`+`
	`540`	`+ // Process Header and Body`
`540`	`541`	`if (part.OriginalContent.Length >= headerEnd)`
`541`	`542`	`{`
`542`	`543`	`ParseHeaderFields(part, headerEnd);`