Commit Graph

13 Commits

Author SHA1 Message Date
Reiley Yang 2e334df8ae
Dedupe IsPackable (#3398) 2022-06-22 19:05:47 -07:00
Cijo Thomas 733c59ad1e
Remove net461 from console inmemory exporters, asp.net (#3193) 2022-04-15 12:11:04 -07:00
Travis Illig d55de344f1
Fix IDE0090: 'using' statement can be simplified. (#3017) 2022-03-10 21:32:41 -08:00
Travis Illig 2a97920ff0
Fix IDE0090: 'new' expression can be simplified. (#3012)
* Set rule for new() usage.

* Fix IDE0090 "'new' expression can be simplified"
2022-03-10 08:57:53 -08:00
Travis Illig 5255917581
Improve VS Code and cross-platform dev experience (#2997) 2022-03-09 09:35:30 -08:00
Michael Maxwell 83fa9fde68
General code improvements (#2744) 2021-12-13 09:42:28 -08:00
Michael Maxwell c60dc3697c
Delete unused `using` statements (#2411) 2021-09-27 10:07:46 -07:00
Cijo Thomas f82bec4497
Disable BaggageRestore tests until stable (#2384) 2021-09-17 18:17:06 -07:00
Mikel Blanchard 2df1a62984
Lost Baggage: Fix TelemetryHttpModule losing Baggage when switching from native to managed threads (#2314)
* WIP working on reproducing and fixing lost baggage.

* TelemetryHttpModule now restores Baggage.

* Updated public api.

* Unit tests.

* Use SuppressFlow for more accurate test.

* CHANGELOG update.

* Attempting to fix unstable test.

* Attempting to fix unstable test #2.
2021-09-15 11:04:50 -07:00
Mikel Blanchard 69c6bf2de9
ASP.NET & ASP.NET Core: Clear baggage when root activity is complete (#2284)
* Clear baggage when root activity is complete.

* Test fixes.

* Code review.

* A couple improvements found reviewing baggage code.
2021-08-30 12:46:01 -07:00
Mikel Blanchard d9ad092dd3
Merge to main: New design for TelemetryHttpModule using ActivitySource + OpenTelemetry.API (#2270)
* New design for TelemetryHttpModule using ActivitySource + OpenTelemetry.API. (#2249)

* New design for TelemetryHttpModule using ActivitySource + OpenTelemetry.API part 2 (#2254)

* Use a single context.Items key for state management to make things more efficient.

* Added a comment for clarity.

* Code review.

* New design for TelemetryHttpModule using ActivitySource + OpenTelemetry.API part 3 (#2256)

* Update ASP.NET instrumentation to use the new TelemetryHttpModule.

* Fixed TelemetryHttpModule not starting its Activity objects. Added an example of request suppression.

* Tweaks an logging improvements.

* Sealed AspNetInstrumentationEventSource.

* Code review.

* New design for TelemetryHttpModule using ActivitySource + OpenTelemetry.API part 4 (#2258)

* Fixed up TelemetryHttpModule unit tests.

* Added tests for the new HasStarted helper and added checks for StartedButNotSampledObj when not sampled.

* Code review.

* New design for TelemetryHttpModule using ActivitySource + OpenTelemetry.API part 5 (#2261)

* Updated ASP.NET instrumentation tests for new TelemetryHttpModule.

* Added a test for the new RecordException option.

* Code review.

* New design for TelemetryHttpModule using ActivitySource + OpenTelemetry.API part 6 (#2264)

* CHANGELOG & README updates.

* Apply suggestions from code review

Co-authored-by: Reiley Yang <reyang@microsoft.com>

Co-authored-by: Reiley Yang <reyang@microsoft.com>

* Lint + sanity checks.

* Lint attempt 2.

* Restored CHANGELOG changes lost in merge.

Co-authored-by: Reiley Yang <reyang@microsoft.com>
2021-08-25 11:41:10 -07:00
Mikel Blanchard dc28d6b9a7 AspNet.TelemetryHttpModule public api + test fixes + renames (#2240) 2021-08-10 20:50:27 -07:00
Mikel Blanchard ec118d4fde Rename Microsoft.AspNet.TelemetryCorrelation -> OpenTelemetry.Instrumentation.AspNet.TelemetryHttpModule. Added project to the solution + updated references & namespaces. (#2238) 2021-08-10 20:50:27 -07:00