Skip to content

feat(logging): Do not capture errors from LoggingIntegration to Sentry by default #4300

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Updated tests

cd10f0f
Select commit
Loading
Failed to load commit list.
Merged

feat(logging): Do not capture errors from LoggingIntegration to Sentry by default #4300

Updated tests
cd10f0f
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Apr 15, 2025 in 0s

83.57% (+0.02%) compared to a610066

View this Pull Request on Codecov

83.57% (+0.02%) compared to a610066

Details

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 83.57%. Comparing base (a610066) to head (cd10f0f).

✅ All tests successful. No failed tests found.

Additional details and impacted files
@@              Coverage Diff               @@
##           potel-base    #4300      +/-   ##
==============================================
+ Coverage       83.55%   83.57%   +0.02%     
==============================================
  Files             144      144              
  Lines           14613    14613              
  Branches         2324     2324              
==============================================
+ Hits            12210    12213       +3     
+ Misses           1686     1684       -2     
+ Partials          717      716       -1     
Files with missing lines Coverage Δ
sentry_sdk/integrations/logging.py 86.98% <100.00%> (+0.68%) ⬆️

... and 3 files with indirect coverage changes