Skip to content

Emit stack trace when TrackExceptionsAsExceptionTelemetry is set to f…#2065

Merged
cijothomas merged 7 commits intomicrosoft:developfrom
RamjotSingh:ramjsing/develop-EmitStackTraceInTrace
Sep 22, 2020
Merged

Emit stack trace when TrackExceptionsAsExceptionTelemetry is set to f…#2065
cijothomas merged 7 commits intomicrosoft:developfrom
RamjotSingh:ramjsing/develop-EmitStackTraceInTrace

Conversation

@RamjotSingh
Copy link
Copy Markdown
Member

Fix Issue #2064.

Changes

ILogger logs StackTraces in TraceTelemetry when TrackExceptionsAsExceptionTelemetry flag is set to false.

Checklist

  • I ran Unit Tests locally.
  • CHANGELOG.md updated with one line description of the fix, and a link to the original issue if available.

The PR will trigger build, unit tests, and functional tests automatically. Please follow these instructions to build and test locally.

Notes for authors:

  • FxCop and other analyzers will fail the build. To see these errors yourself, compile localy using the Release configuration.

Notes for reviewers:

  • We support comment build triggers
    • /AzurePipelines run will queue all builds
    • /AzurePipelines run <pipeline-name> will queue a specific build

@RamjotSingh
Copy link
Copy Markdown
Member Author

/AzurePipelines run

@azure-pipelines
Copy link
Copy Markdown

Commenter does not have sufficient privileges for PR 2065 in repo microsoft/ApplicationInsights-dotnet

@eddynaka
Copy link
Copy Markdown
Contributor

/AzurePipelines run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 3 pipeline(s).

Comment thread LOGGING/src/ILogger/ApplicationInsightsLogger.cs Outdated
Comment thread LOGGING/test/ILogger.Tests/ILoggerIntegrationTests.cs
Copy link
Copy Markdown
Contributor

@cijothomas cijothomas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.
Left a small comment in testing.

@cijothomas
Copy link
Copy Markdown
Contributor

/AzurePipelines run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 3 pipeline(s).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants