You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by "Bart Laarhoven (JIRA)" <ji...@apache.org> on 2012/12/19 17:51:13 UTC

[jira] [Updated] (EMAIL-123) Misleading exception message

     [ https://issues.apache.org/jira/browse/EMAIL-123?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Bart Laarhoven updated EMAIL-123:
---------------------------------

    Description: 
I'm not too experienced with filing bug reports and I don't know how to make a patch file, so I'll just do it like this. I also don't know whether this is fixed already in an unreleased version... But I just hope I can help.

On line 1240 of Email.java in version 1.2 of Commons Email, the message of the exception is extended with this.getSmtpPort. This is incorrect; this should be something like:

(this.isSSL()?this.getSslSmtpPort():this.getSmtpPort())

Else it results in questions like http://stackoverflow.com/questions/13951322/javamail-email-commons-empty-message-gives-strange-error
;-)

  was:
I'm not too experienced with filing bug reports and I don't know how to make a patch file, so I'll just do it like this. I also don't know whether this is fixed already in an unreleased version... But I just hope I can help.

On line 1240 of Email.java in version 1.2 of Commons Email, the message of the exception is extended with this.getSmtpPort. This is incorrect; this should be something like:

(this.isSSL()?this.getSslSmtpPort():this.getSmtpPort())

Else there are questions like http://stackoverflow.com/questions/13951322/javamail-email-commons-empty-message-gives-strange-error
;-)

    
> Misleading exception message
> ----------------------------
>
>                 Key: EMAIL-123
>                 URL: https://issues.apache.org/jira/browse/EMAIL-123
>             Project: Commons Email
>          Issue Type: Bug
>    Affects Versions: 1.2
>            Reporter: Bart Laarhoven
>              Labels: patch
>
> I'm not too experienced with filing bug reports and I don't know how to make a patch file, so I'll just do it like this. I also don't know whether this is fixed already in an unreleased version... But I just hope I can help.
> On line 1240 of Email.java in version 1.2 of Commons Email, the message of the exception is extended with this.getSmtpPort. This is incorrect; this should be something like:
> (this.isSSL()?this.getSslSmtpPort():this.getSmtpPort())
> Else it results in questions like http://stackoverflow.com/questions/13951322/javamail-email-commons-empty-message-gives-strange-error
> ;-)

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira