Improved address parser to handle unbalanced ')' (rfc7103)

Author: jstedfast

MimeKit/Utils/ByteExtensions.cs

@@ -145,6 +145,9 @@ static ByteExtensions ()
 			// Note: Allow '[' and ']' in the display-name of a mailbox address
 			table['['] |= CharType.IsPhraseAtom;
 			table[']'] |= CharType.IsPhraseAtom;
+
+			// Note: Allow ')' in the display-name of a mailbox address for rfc7103 unbalanced parens
+			table[')'] |= CharType.IsPhraseAtom;
 		}
 
 		//public static bool IsAscii (this byte c)

UnitTests/InternetAddressListTests.cs

@@ -900,6 +900,31 @@ public void TestParseErrantComma ()
 			AssertParseAndTryParse (text, encoded, expected);
 		}
 
+		[Test]
+		[Ignore ("Address parser consumes the entire string as an unterminated comment")]
+		public void TestParseMailboxWithUnbalancedOpenParenthesis ()
+		{
+			const string text = "(Testing <[email protected]>";
+			const string encoded = "Testing <[email protected]>";
+			var expected = new InternetAddressList {
+				new MailboxAddress ("Testing", "[email protected]")
+			};
+
+			AssertParseAndTryParse (text, encoded, expected);
+		}
+
+		[Test]
+		public void TestParseMailboxWithUnbalancedClosedParenthesis ()
+		{
+			const string text = "Testing) <[email protected]>";
+			const string encoded = "\"Testing)\" <[email protected]>";
+			var expected = new InternetAddressList {
+				new MailboxAddress ("Testing)", "[email protected]")
+			};
+
+			AssertParseAndTryParse (text, encoded, expected);
+		}
+
 		[Test]
 		public void TestParseMailboxWithUnbalancedQuotes ()
 		{