getsentry / getsentry/sentry-dotnet

Enable logging by default [.NET]

Open
#5,183 2 comments 0 reactions 0 assignees View on GitHub
.NET Breaking Change Logs Next Major State: Needs Discussion Task
Dominant language
C#
Stars
770
Forks
248
Avg merge
3d 4h
Merged PRs (30d)
49

Description

Update logs behavior to follow the same opt-in model as `enableMetrics`:

* Users are considered opted-in when:
* They call the equivalent of `Sentry.logger.*`, or
* They explicitly enable a logging integration.
* Change default: set enable logs to `true`.
* Audit all logging integrations to ensure:
* No logs are emitted unless the integration is explicitly enabled.
* Update documentation:
* Explain new default behavior of `enableLogs`.
* Clarify requirement to explicitly enable logging integrations.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.