-
Notifications
You must be signed in to change notification settings - Fork 848
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
Solves #767 HttpTraceContext should not throw exceptions for non-fatal cases #782
Solves #767 HttpTraceContext should not throw exceptions for non-fatal cases #782
Conversation
looks like you have some failing tests. |
Codecov Report
@@ Coverage Diff @@
## master #782 +/- ##
============================================
- Coverage 78.58% 78.47% -0.12%
Complexity 778 778
============================================
Files 101 101
Lines 2779 2787 +8
Branches 268 269 +1
============================================
+ Hits 2184 2187 +3
- Misses 493 498 +5
Partials 102 102
Continue to review full report at Codecov.
|
opentracing_shim/src/main/java/io/opentelemetry/opentracingshim/Propagation.java
Outdated
Show resolved
Hide resolved
context_prop/src/main/java/io/opentelemetry/context/propagation/HttpTextFormat.java
Outdated
Show resolved
Hide resolved
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.
looks good to me.
No description provided.