[EMAIL-123] Misleading exception message

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Duplicate
• Affects Version/s: 1.2
• Fix Version/s: 1.3
• Labels: patch
• Assignee: Unassigned
• Reporter: Bart Laarhoven
• Created: 19/Dec/12 16:34
• Updated: 11/Jan/13 15:47
• Resolved: 05/Jan/13 20:41

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

Issue Links

• duplicates: EMAIL-91 Incorrect SMTP Port number shown in error message when an email fails to send due to a blocked port and SSL is used.