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.

Sign up for GitHub

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

Jump to bottom

Update JAX-RS to 2.1 #52

Open
hypnoce opened this issue Nov 16, 2017 · 3 comments
Open

Update JAX-RS to 2.1 #52

hypnoce opened this issue Nov 16, 2017 · 3 comments

Comments

@hypnoce
Copy link
Contributor

hypnoce commented Nov 16, 2017

Hi,

as requested in #49, JAX-RS was not updated to the latest version 2.1.
Do you think it brings value to do so ?

Thanks !

Francois
@tmszdmsk
Copy link
Contributor

@hypnoce
IMHO if there are no features we want to use in JAX-RS integration and it is backwards compatible (it's sane to assume it is) then there is no benefit of doing so. It can accidentally bump the version of JAX-RS in projects using it when resolving version.

@hypnoce
Copy link
Contributor Author

hypnoce commented Jan 31, 2018

Indeed it is premature to do so now.

Nevertheless, the jaxrs dependency is declared as provided. Therefore there should not be any bump accident.

I think it makes more sense to work on this issue when upgrading the test dependencies to validate this instrumentation to newer versions of the jaxrs implementations.

@jpkrohling
Copy link
Contributor

Anyone up to speed with the current state of JAX-RS ? Is there anything that we'd need to do, or can this one be closed?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@jpkrohling @tmszdmsk @hypnoce