Cannot parse messages without a session #710

[jmehrens]

Signed-off-by: jmehrens jason_mehrens@hotmail.com

[jmehrens]

Per @bshannon (R.I.P) on What is the use of javax.mail.Session?:

To add to that... The support for a null Session is there mostly to provide some reasonable behavior in the case where people screw up and fail to create a Session.

Classic! Point being is that null Session is allowed in MimeMessage the fallback is to use the default session when needed. The approach to this patch is to mimic the behavior in Transport.send0 which is:

Session s = (msg.session != null) ? msg.session :
                Session.getDefaultInstance(System.getProperties(), null);

The alternate patch would be to call StreamProvider.provider() in case the session is null. I think this is more consistent with what is already done.

[lukasj]

@jmehrens I'll respin 2.1.3 so this gets there. Do you have anything else for 2.1.3 before I do that?

[jmehrens]

@lukasj The only things that might be worth while are a backport of:

1. Illegal reflective access by com.sun.mail.util.SocketFetcher eclipse-ee4j/angus-mail#124 to v1.x
2. MailHandler should catch ServiceConfigurationError #123 eclipse-ee4j/angus-mail#125 to 1.x