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

Java 9: Illegal reflective access warning caused by JAXB dependency #23

Closed
liorhar opened this issue Feb 22, 2021 · 8 comments
Closed
Assignees

Comments

@liorhar
Copy link

liorhar commented Feb 22, 2021

When using postmark's lib with java version greater than 8 there's a warning about illegal access caused by the jaxb dependency.
See here - javaee/jaxb-v2#1197

Upgrading from version 2.3.0 to 2.3.1 should resolve that issue.
Btw, does postmark use jaxb at all? I excluded it from my dependencies and everything seems to work fine

@ibalosh ibalosh self-assigned this Feb 24, 2021
@ibalosh
Copy link
Contributor

ibalosh commented Mar 2, 2021

Hi @liorhar

could you share details which version of Java are you using? Jackson library requires those dependencies and can cause issues on newer java versions when removed.

Igor

@liorhar
Copy link
Author

liorhar commented Mar 2, 2021

I'm using java 13 (adopt openjdk) I believe the issue is with every version greater than 8

@ibalosh
Copy link
Contributor

ibalosh commented Mar 9, 2021

Hi @liorhar

I have investigated this issue and need bit more help to nail the issue. It seems to be a warning happening only for patch requests, which would be if using: "setMessageStream" method. Do you see it on any other endpoints?

@liorhar
Copy link
Author

liorhar commented Mar 9, 2021

hi @ibalosh I only use the method deliverMessageWithTemplate and that's where I saw the warning.

@ibalosh
Copy link
Contributor

ibalosh commented Mar 9, 2021

gotcha

@ibalosh
Copy link
Contributor

ibalosh commented Mar 10, 2021

Hi @liorhar

we have released a new version of the library (1.7) , which uses later version of Jersey client in which jaxb is optional and removed. Could you let me know if you still see the warning, and if so, could you share the exact warning here?

Thanks

@liorhar
Copy link
Author

liorhar commented Mar 14, 2021

yeah, it looks good now, thanks 🙏

@ibalosh
Copy link
Contributor

ibalosh commented Mar 14, 2021

Hey, glad it works now ok, thank you for letting us know

@ibalosh ibalosh closed this as completed Mar 14, 2021
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

No branches or pull requests

2 participants