-
Notifications
You must be signed in to change notification settings - Fork 713
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
Jersey 2.x support #38
Conversation
<groupId>com.github.kristofa</groupId> | ||
<artifactId>brave-impl-spring</artifactId> | ||
<version>2.4-SNAPSHOT</version> | ||
<scope>provided</scope> |
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.
scope test
Thanks for contributing Jersey 2 support! |
Hi @ctroullis, some people have added resteasy 3 support and also changed the jersey2 module. If you would have time, could you see if their changes still work for you? See pull request #48. Thank you! |
Sure, I'll check it out. I may not be able get to it until Monday though. |
No problem. Thanks a lot! |
This pull request is part of brave 2.4 which is just released and should become available through Maven Central in a few hours. |
I needed to port the Jersey module over to use Jersey 2.x, and I saw it was on the contribution ideas list so I thought I'd clean it up and submit it. I tried to keep it as consistent as I could with the existing Jersey module. I made it a separate module to avoid classpath issues between Jersey 1.x and 2.x.
There are a couple of classes (like TraceData) that I could use as is and I copied them over, as I wasn't sure where would be a good common place to put classes that are used by multiple modules.