* If an outbound HttpRequestMessage does not include the traceparent header then don't throw.
Made the header values getters and setter static to avoid the Func and Action allocations per invocation.
* Missed a getter usage in HttpInListener
* Extending an existing unit test to validate the new HttpRequestMessage header value getter.
Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
* Added options to HttpWebRequest instrumentation.
* Cleaned up some usings.
* Added IsInjected to ITextFormatActivity.
* Switched the Http instrumentation filter back to internal so we can move ahead with the other stuff while the API is reviewed.
* Updated HttpClient instrumentation to check IsInjected instead of traceparent header directly.
* FilterFunc is back to public. Removed ctors from http options.
* Removed ctor that isn't needed.
Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
* Replace references to SpanAttributeConstants with constants from SemanticConventions
* Missed a few references to the "component" attribute in some tests.
Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
* Attempting to get Redis instrumentation up and working using ActivitySource API.
* updating tests
* renaming and updating tests
* Test fixes.
* Fixed Redis net461 tests failing on connection.
* Redis integration tests and bug fixes.
* Adding tests
* Put back redis connection options.
* Updates for changes in master.
* adding comments to testRedis
* updating based on comments
* updating summary
* Switched a couple spots using string keys to constants.
* Redis integration tests
* Small tweak to the GitHub action to make it more consistent with the others.
* Made instrumentation internal. Updated README.
* rename MaxFlushInterval to FlushInterval, adding flushInterval to samples
* Removed running of non-integration tests from Redis dockerfile.
Co-authored-by: Eddy Nakamura <eddynaka@gmail.com>
Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
* Added instrumentation for netfx SqlClient.
* Code review.
* Send "db.statement_type" instead of "db.statementType".
* Added integration tests for SqlEventSource.
* Adding .github folder to solution.
* Added a couple missing files into the solution.
* Added FakeSqlEventSource tests.
* Fixed up GrpcClientDiagnosticListener for EventSource changes.
* Removed extra ActivityListener from unit tests.
Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
* Pull logic out of gRPC client instrumentation into helper class + add tests
* Fix parsing rpc.service with package that has multiple parts
Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
moving folder
updating test to Tests.dll and folder to get less files
updating folder again
adding other versions to test
Adding linux and renaming
updating ubuntu version
updating yml linux
adding variables and using matrix
adding strategy to matrix
updating variables
new test
updating variables
updating variables
updating test packages
trying to add cache option
testing cache on windows
removing cache folder and trying to optmize files
updating
adding more workflows to test parallel
updating windows one job pipeline
updating
renaming
updating
updating test
adding verbose and breaking one test to double check
reverting test, adding new line, removing verbose from parallel
Update dotnet-core.yml
Update dotnet-core.yml
updating fetch depth
fetching all
Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
* expose ActivitySourceFake within OT which mimics ActivitySource for activities created using without ActivitySource.
* modify all instrumentation to use the ActivitySourceAdapter
* minor todo coment
* build fix
* Fix empty activitysource listening for asp.net
* Enable .NET 452 - part 01
* removing warning and adding net452 to shims.OpenTracing
* adding net452 condition to dependencies code
* undoing non-related changes to pull request
* adding sytem.tuple to dependencies.test for net 452. changing condition to netframework
Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
* Adds HostingProject extension method to support new Activity based api.
* pr comment
* some test refactoring for better cleanup to ensure tests dnt interfere with each other
* use the same activity created by existing instrumentation. Simply enhance it.
* remove sampling from instrumentation
* add asp.net core instrumentation
* move sampling to Instrumentation for now.
Make Asp.Net instrumentation work with Activity API
* Add httpClient .net core instrumentation
* Ad SqlClientInstrumentation
* remove sqlclientinstrumentation from previous instrumentation
* Quick implementation for AzureClients - not validated as there are unit tests and this is not planned to be in this repo as well.
* fix examples
* fix sampling flag
* made sample app work with jaeger
* Mark todos and fix AspNet tests
* Fix asp.net core tests and mark TODOs
* Add TODO for httpclient .net core test and fix test
* add todo and fix httpclient test
* add todos and fil sqlclienttests
* Make OpenTelemetrySDK disposable and take care of disposing all ds subscriptions.
* Added OpenTelemetry.Default instead of static method
* AspNet, AspNetCore fix Dispose issue
* stylecop stuff lost i merge
* Enable StyleCop rules for Test projects
This enables most of the StyleCop for test projects cleaning up any
errors as appropriate.
* Enable StyleCop rules for Test projects
This enables most of the StyleCop for test projects cleaning up any
errors as appropriate.
* Add line break at end of ruleset files
* Correct multi-line parameter after rebase
* Updated HttpWebRequestDiagnosticSource to use the new .NET 5 ActivitySource API.
* Code review part 1 and string allocation reduction.
* Code review part 2 added ExceptionInitializingInstrumentation event.
* Code review feedback 3.
* Code review feedback 4.
* Updated unit tests and some code review feedback.
* Updated to JaegerActivityExporter since it was just merged.
* Refactored http tag value caching so that it can be shared in dependencies project.
* Code review.
* Noticed a couple comments needed to be updated.
* Added Activity.IsAllDataRequested logic.
* Code review.
* Reverted change to OpenTelemetrySdk.
Co-authored-by: Cijo Thomas <cithomas@microsoft.com>
Cijo Thomas