-
Notifications
You must be signed in to change notification settings - Fork 97
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
StreamProvider not static and OSGi support #579
Conversation
b70c1d1
to
6e260e2
Compare
6e260e2
to
70bebf1
Compare
Signed-off-by: Jorge Bescos Gascon <jorge.bescos.gascon@oracle.com>
70bebf1
to
b4cd7bb
Compare
Signed-off-by: Jorge Bescos Gascon <jorge.bescos.gascon@oracle.com>
Class<T> clazz = null; | ||
try { | ||
if (classLoader == null) { | ||
clazz = (Class<T>) Class.forName(className); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
unchecked cast... add appropriate suppresswarnings....
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
with activation-api 2.1.0-RC2 needs angus-activation at test scope (or some dummy impl probably...) to build
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll fix outstanding minor things myself in the next PR
Causes downstream problems for Jenkins: #665 |
StreamProvider is supposed to be implemented by different implementations, and it was instanced one time in a static variable.
This could be a problem in server containers, because only one instance would exist. It could happen that it is loaded as com.sun.mail, and later an application with other implementation is deployed. In this case, this application would still use com.sun.mail.