commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Bart Laarhoven (JIRA)" <j...@apache.org>
Subject [jira] [Updated] (EMAIL-123) Misleading exception message
Date Wed, 19 Dec 2012 16:51:13 GMT

     [ 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

Mime
View raw message