[ 
https://issues.apache.org/jira/browse/MIME4J-73?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12631505#action_12631505
 ] 

Robert Burrell Donkin commented on MIME4J-73:
---------------------------------------------

I think I agree. Please contribute a patch :-)

> IllegalArgumentException in Message constructor when parsing erroneous message
> ------------------------------------------------------------------------------
>
>                 Key: MIME4J-73
>                 URL: https://issues.apache.org/jira/browse/MIME4J-73
>             Project: JAMES Mime4j
>          Issue Type: Bug
>    Affects Versions: 0.5
>            Reporter: Markus Wiederkehr
>
> 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.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to