-
Notifications
You must be signed in to change notification settings - Fork 290
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
Owin instrumentation #156
Owin instrumentation #156
Conversation
Codecov Report
@@ Coverage Diff @@
## main #156 +/- ##
==========================================
+ Coverage 78.62% 78.78% +0.15%
==========================================
Files 79 85 +6
Lines 1984 2102 +118
==========================================
+ Hits 1560 1656 +96
- Misses 424 446 +22
|
@SergeyKanzhelev Any free cycles to do a review for me? Trying not to bug the guys working hard to finish up metrics support in SDK. |
src/OpenTelemetry.Contrib.Instrumentation.Owin/Implementation/OwinInstrumentationEventSource.cs
Show resolved
Hide resolved
src/OpenTelemetry.Contrib.Instrumentation.Owin/OwinInstrumentationOptions.cs
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.
LGTM. Just one question about the Propagators
option.
…opentelemetry-dotnet-contrib into owin-instrumentation
Resolves #56
Changes
Added OpenTelemetry OWIN Instrumentation. Started from open-telemetry/opentelemetry-dotnet#1472 but made the following changes:
TODOs