Skip to content

Conversation

lcian
Copy link
Member

@lcian lcian commented Sep 25, 2025

Description

This enables log by default if the crate is used with the logs feature flag.
Otherwise, the user would have to enable the feature flag and then set enable_logs on top.


Note

Sets ClientOptions::enable_logs default to true under the logs feature and documents the behavioral change.

  • Core:
    • Set sentry-core/src/clientoptions.rs ClientOptions::enable_logs default to true (when logs feature is enabled).
  • Docs/Changelog:
    • Add Unreleased entry documenting the new default and that tracing/log integrations send structured logs at ≥ INFO by default when logs is enabled.

Written by Cursor Bugbot for commit a4aa065. This will update automatically on new commits. Configure here.

Copy link

codecov bot commented Sep 25, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 73.71%. Comparing base (d9cdf34) to head (a4aa065).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #910      +/-   ##
==========================================
+ Coverage   73.46%   73.71%   +0.25%     
==========================================
  Files          64       64              
  Lines        7518     7518              
==========================================
+ Hits         5523     5542      +19     
+ Misses       1995     1976      -19     

@lcian lcian marked this pull request as ready for review September 30, 2025 11:45
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.

2 participants