[GitHub] [commons-email] michael-o commented on issue #8: EMAIL-184: Should throw EmailException if email provided is null

2019-12-26 Thread GitBox
michael-o commented on issue #8: EMAIL-184: Should throw EmailException if email provided is null URL: https://github.com/apache/commons-email/pull/8#issuecomment-569106066 If some of the argument must not be blank/null, it should be handled by classes like this:

[GitHub] [commons-email] michael-o commented on issue #8: EMAIL-184: Should throw EmailException if email provided is null

2019-12-26 Thread GitBox
michael-o commented on issue #8: EMAIL-184: Should throw EmailException if email provided is null URL: https://github.com/apache/commons-email/pull/8#issuecomment-569105894 I am not fully convinced here. What happens with your patch when you provide `null` for mail?