Skip to content
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

Proposed workaround to the jakarta.mail.util.StreamProvider #702

Closed
wants to merge 1 commit into from

Conversation

vionta
Copy link

@vionta vionta commented Nov 27, 2023

Currently, when the instantiation of the StreamProvider Implementation fails using the Thread Context Class Loader, there's no fallback to the newInstance on the App Class Loader.
This is a proposed workaround, that allows to load the StreamProvider implementation class indicating the explicit class name (Declaration example under this line):
System.setProperty("jakarta.mail.util.StreamProvider","org.eclipse.angus.mail.util.MailStreamProvider");

@jmehrens
Copy link
Contributor

jmehrens commented Dec 7, 2023

I think I have this case covered in #701

@vionta
Copy link
Author

vionta commented Dec 9, 2023

Hi, I made a specific test on the environment and code from #701 seems to work.

@jmehrens
Copy link
Contributor

@vionta That is great news! I'm holding off on merging #701 until I complete the matching PR for Activation.

@jmehrens jmehrens closed this Jan 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants