IllegalArgumentException when validating email address ending with "High Octet Preset" control character

[rohannagar]

IllegalArgumentException when validating email address ending with "High Octet Preset" control character

[PascalSchumacher]

@PascalSchumacher Good point. OutOfMemoryError actually would not be caught since it extends Error. If we did something like catch (Throwable t), then it would be caught.

Yes, you are right. I do not know what I was thinking. Sorry!

Thank you very much for the new release! 👍 😃

[RohanNagar]

Thank you for the report @PascalSchumacher. You're right that this is not the expected behavior, I believe JMail should return false in this case as this would not be considered a valid email address.

[RohanNagar]

@PascalSchumacher please let me know if the fix in fbeecce looks good to you and I will publish a new version v1.3.3.

[PascalSchumacher]

@RohanNagar Thanks for the quick fix. I always a bit nervous when I see a catch block catching Exception because it also catches Throwables like OutOfMemoryError. According to the Java Doc this method only throws IllegalArgumentExceptions so I believe catching that would be better. What do you think?

[RohanNagar]

@PascalSchumacher Good point. OutOfMemoryError actually would not be caught since it extends Error. If we did something like catch (Throwable t), then it would be caught.

However since this method only throws IllegalArgumentException it makes sense to only catch that. I've updated and released version 1.3.3, which should be available on maven central in about 10 minutes!