-
Notifications
You must be signed in to change notification settings - Fork 775
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
Auto-instrumentation for JaxWS #2176
Conversation
Please update README as well |
group = "javax.xml.ws" | ||
module = "jaxws-api" | ||
versions = "[2.0,]" | ||
skipVersions += '2.1-1' // contains broken dependency |
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.
Does it make sense to add assertInverse = true
?
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.
Not sure it is needed. The API is very stable, even version 2.0 is from 2006 or something. Current is 2.3. And I couldn't find 1.0. Then it all moved to jakarta and in November 2020 they released new versions of all java EE components (Java EE 9), where packages were changed to jakarta...
instead of javax...
. So all existing Java EE instrumentations will have to be revisited to support Java EE 9 with new package names...
JaxWS is a set of APIs for SOAP WebServices. This PR adds auto-instrumentation for
@javax.jws.WebService
annotation from Java Web Services Metadata (separate library, but jaxws assumes it is present) used to define a web-service,javax.xml.ws.Provider
interface from Jax WS API, alternative way of handling SOAP messages.