[MIME4J-73] IllegalArgumentException in Message constructor when parsing erroneous message

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 0.4
• Fix Version/s: 0.5
• Assignee: Unassigned
• Reporter: Markus Wiederkehr
• Created: 16/Sep/08 14:33
• Updated: 04/Feb/09 20:50
• Resolved: 20/Sep/08 18:42

Description

The constructor of Message(InputStream) uses its inner class MessageBuilder to parse the input stream and create the object. The method MessageBuilder.field(String) calls Field.parse(String) to parse header fields. This method throws an IllegalArgumentException on parse errors.

Clearly the Message constructor should throw a MimeException instead of an IllegalArgumentException if an invalid header field is encountered (or else it should simply ignore the field). So either Field.parse(String) should throw a MimeException or MessageBuilder.field(String) should catch and convert it.

And by the way, could you make MessageBuilder public so it can be extended if a slightly different behavior is desired.

Issue Links

• relates to: MIME4J-77 Decide Whether MimeException should extend IOException