jakartaee / jakartaee/mail-api

Improve JavaDoc Session.getInstance(Properties, Authenticator)

Open
#831 1 comment 0 reactions 1 assignee Claimed by @jmehrens View on GitHub
bug
Dominant language
Java
Stars
285
Forks
109
Avg merge
15h 19m
Merged PRs (30d)
1

Description

**Describe the bug**
The class loading of providers changed if an Authenticator parameter is set for the call of `Session.getInstance(Properties, Authenticator)`. This is not documented and completely unexpected. It has cost me days to understand why with OAuth and Authenticator in production the provider was not loaded.

```
Class cl;
if (authenticator != null) {
cl = authenticator.getClass();
} else {
// Use implementation class, because that class loader has access to jakarta.mail module and implementation resources
cl = streamProvider.getClass();
}
```
**Expected behavior**
The same class loading behavior with and without Authenticator. The minimum is to documented this behavior.
For consistence with other parts of the API the context class loader should be used.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.