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

Add native Jersey 2.x instrumentation #611

Closed
codefromthecrypt opened this issue Feb 18, 2018 · 1 comment
Closed

Add native Jersey 2.x instrumentation #611

codefromthecrypt opened this issue Feb 18, 2018 · 1 comment

Comments

@codefromthecrypt
Copy link
Member

Currently, people use JAX-RS with Jersey. However, this spec is lacking the ability to get details such as the http route (#602). A direct integration can do this, as evidenced in micrometer:

https://github.com/micrometer-metrics/micrometer/tree/master/micrometer-jersey2

Support for this would also help our dropwizard instrumentation make use of higher fidelity data (DW doesn't need the JAX-RS abstraction as it always uses jersey)

cc @henrikno @tliuliaison @jplock @mweirauch @suls @jkschneider
@codefromthecrypt codefromthecrypt mentioned this issue Feb 18, 2018
Merged
codefromthecrypt pushed a commit that referenced this issue Feb 20, 2018
Adds Jersey 2.x server tracing support
f4edad2 
This uses an application event listener to do a better job tracing
Jersey servers than what can be done with JAX-RS. For example, the event
listener can handle errors natively, and act more precisely with regards
to jersey-specifics like `ManagedAsync`

Fixes #611
@codefromthecrypt codefromthecrypt mentioned this issue Feb 20, 2018
Merged
@codefromthecrypt
Copy link
Member Author

#614

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
1 participant
@codefromthecrypt