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

Make javax.xml.soap. dynamically accessed to work around Java 9 issues #1243

Closed
cowtowncoder opened this issue May 21, 2016 · 1 comment
Closed
Milestone

Comments

@cowtowncoder
Copy link
Member

Looks like Java 9 by default provides smaller set of classes, and one case where this affects Jackson directly (as per unit tests) is lack of esoteric type javax.xml.soap.Detail.
Since I can't see anything explicit under src/main I suspect it's a test artifact; nonetheless test shouldn't fail on Java 9 as it does currently: either test should dynamically see if class is available or just remove.

@cowtowncoder
Copy link
Member Author

Looks like there is no explicit support for javax.xml.soap.Detail, and the origin of the broken test (TestSOAP) is bit of a mystery; further, test really does not test output wrt validity. So I think I will just go ahead and remove it so as not to fail on Java 9. That seems like the best course of action on short term.

@cowtowncoder cowtowncoder added this to the 2.8.0 milestone May 26, 2016
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

1 participant