⚠ This page is served via a proxy. Original site: https://github.com
This service does not collect credentials or authentication data.
Skip to content

Conversation

@drazisil-codecov
Copy link
Contributor

@drazisil-codecov drazisil-codecov commented Jan 9, 2026

Summary

Removes sentry_sdk.capture_exception() calls for ReportEmptyError and ReportExpiredException - these are expected user input validation issues that don't need to be tracked as errors in Sentry.

Problem

These exceptions were being sent to Sentry creating ~150K+ error events per week:

  • ReportEmptyError: ~129K events/week (customer uploaded empty/invalid report)
  • ReportExpiredException: ~24K events/week (customer uploaded expired report)

These are already:

  • ✅ Properly logged (warning/info level)
  • ✅ Handled gracefully (return error result to caller)
  • ✅ Communicated back to customer

Solution

Remove the unnecessary capture_exception() calls while keeping all logging and error handling intact.

Impact

  • Reduces Sentry noise by ~150K events/week
  • No change to customer experience
  • No change to debugging capability (logs are preserved)

Note

Removes Sentry exception capture for expected user-input errors in report processing, keeping logging and error handling intact.

  • In build_report_from_raw_content, removed sentry_sdk.capture_exception() for ReportExpiredException and ReportEmptyError
  • Maintains existing log.info/log.warning messages and returns appropriate ProcessingError results for these cases

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

@drazisil-codecov drazisil-codecov requested a review from a team January 9, 2026 16:28
ReportEmptyError and ReportExpiredException are expected user input
validation issues (customer uploaded empty or expired reports). These
were being sent to Sentry via capture_exception() creating ~150K+ error
events per week, even though they are already properly logged and handled.

This change removes the sentry_sdk.capture_exception() calls for these
expected cases while keeping the warning/info logs for debugging.
@drazisil-codecov drazisil-codecov force-pushed the joebecher/reduce-sentry-noise-expected-errors branch from e0dd840 to 295d1d6 Compare January 9, 2026 16:30
@sentry
Copy link

sentry bot commented Jan 9, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.78%. Comparing base (6915276) to head (295d1d6).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #642      +/-   ##
==========================================
- Coverage   93.78%   93.78%   -0.01%     
==========================================
  Files        1291     1291              
  Lines       46907    46905       -2     
  Branches     1517     1517              
==========================================
- Hits        43991    43989       -2     
  Misses       2607     2607              
  Partials      309      309              
Flag Coverage Δ
workerintegration 59.14% <ø> (+<0.01%) ⬆️
workerunit 91.26% <ø> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@codecov-notifications
Copy link

codecov-notifications bot commented Jan 9, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

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