[CAMEL-13712] If a javax.mail.Session gets referred to using the "session" URL parameter, Apache Camel Mail ignored its hostnames.

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 2.22.1
• Fix Version/s: 3.0.0.RC1, 3.0.0
• Component/s: camel-mail
• Estimated Complexity: Unknown
• Assignee: Claus Ibsen
• Reporter: Philipp Bachmann
• Created: 01/Jul/19 10:53
• Updated: 07/Aug/19 06:58
• Resolved: 07/Aug/19 06:58

Description

A javax.mail.Session injected using JNDI can refer to one host per protocol. Referring to such a session using the URL query string parameter session=#jndiName seems to ignore the hosts the session is bound to:

My first try was simply e.g. "smtp://?session=#jndiName". Now Apache Camel Mail complains about a missing hostname.

So I tried "smtp://localhost?session=#jndiName" expecting that "localhost" would have been overridden using the SMTP host the session refers to, which is not the case. Camel complains that no SMTP server was listening on the host named "localhost".

The workaround that works – but is somehow redundant in my opinion – is:

@Resource(mappedName = "jndiName")
{{ private Session mySession;}}

...to("smtp://"+mySession.getProperty("mail.smtp.host")+"?session=#jndiName")...

The same also applies to the IMAP server (and likely to the other protocols supported by Java Mail as well).

It would be nice iff referring to a JNDI injected session would take the session as-is to simplify the URL.

I found an example for using the "session" parameter: "MailUsingCustomSessionTest". It would be helpful to have another example at hand that either takes the hostname from the session as my example above or that does not contain a hostname at all, given, Apache Camel Mail will support that in the future.