Skip to content

Commit

Permalink
Update profiler skipped assembly prefixes
Browse files Browse the repository at this point in the history
We need to instrument Microsoft.Extensions.Logging.Abstractions now, so can't skip the Microsoft.Extensions prefix

This adds most of them to the skip list, but there may be a better approach, e.g. an additional "don't skip" list?
  • Loading branch information
andrewlock committed Aug 5, 2021
1 parent d6947d2 commit eb5d398
Showing 1 changed file with 16 additions and 1 deletion.
17 changes: 16 additions & 1 deletion src/Datadog.Trace.ClrProfiler.Native/dd_profiler_constants.h
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,22 @@ const WSTRING skip_assembly_prefixes[]{
WStr("Microsoft.ApplicationInsights"),
WStr("Microsoft.Build"),
WStr("Microsoft.CSharp"),
WStr("Microsoft.Extensions"),
WStr("Microsoft.Extensions.Caching"),
WStr("Microsoft.Extensions.Configuration"),
WStr("Microsoft.Extensions.DependencyInjection"),
WStr("Microsoft.Extensions.DependencyModel"),
WStr("Microsoft.Extensions.Diagnostics"),
WStr("Microsoft.Extensions.FileProviders"),
WStr("Microsoft.Extensions.FileSystemGlobbing"),
WStr("Microsoft.Extensions.Hosting"),
WStr("Microsoft.Extensions.Http"),
WStr("Microsoft.Extensions.Identity"),
WStr("Microsoft.Extensions.Localization"),
WStr("Microsoft.Extensions.ObjectPool"),
WStr("Microsoft.Extensions.Options"),
WStr("Microsoft.Extensions.PlatformAbstractions"),
WStr("Microsoft.Extensions.Primitives"),
WStr("Microsoft.Extensions.WebEncoders "),
WStr("Microsoft.Web.Compilation.Snapshots"),
WStr("System.Core"),
WStr("System.Console"),
Expand Down

0 comments on commit eb5d398

Please sign in to comment.