[do not merge] Testing merge - #7704
sentrivana wants to merge 4 commits into
Conversation
- ignore everything except `index.py` and `.gitignore` itself. - avoids having dependencies and files generated by tests showing up as untracked.
### Description Multiple user callback fixes: - Emit client reports with reason `callback_error` ([docs](https://develop.sentry.dev/sdk/telemetry/client-reports/#discard-reasons-1)) when telemetry is dropped because of an error in a user callback - This was missing for `before_send`, `before_send_transaction`, `before_send_metric`, `before_send_log` - Emit client reports with reason `before_send` if a log or a metric is explicitly dropped in `before_send_metric`/`before_send_log` by returning `None` - Isolate `profiles_sampler` errors in `capture_internal_exceptions` - Isolate `error_processor` errors in `capture_internal_exceptions` - Event and error processors used to let the event go through if an error occurred. Now they'll drop the event as per spec. #### Issues Closes https://linear.app/getsentry/issue/PY-2763/adhere-to-user-callback-error-spec
As this is technically a breaking change and we have a major around the corner, let's do it there. Reverting this change on master.
Codecov Results 📊✅ 62404 passed | ❌ 18 failed | ⏭️ 2890 skipped | Total: 65312 | Pass Rate: 95.55% | Execution Time: 174m 11s 📊 Comparison with Base Branch
➕ New Tests (18)View new tests
❌ Failed Tests
|
| File | Patch % | Lines |
|---|---|---|
| sentry_sdk/client.py | 100.00% |
Coverage diff
@@ Coverage Diff @@
## master #PR +/-##
==========================================
- Coverage 90.21% 90.12% -0.09%
==========================================
Files 194 178 -16
Lines 26030 21008 -5022
Branches 9678 7584 -2094
==========================================
+ Hits 23480 18932 -4548
- Misses 2550 2076 -474
- Partials 1454 1199 -255Generated by Codecov Action
No description provided.