chore(bottle): Remove transaction code - #7272
Conversation
832e83e to
46fe396
Compare
4e951ab to
0633083
Compare
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 0633083. Configure here.
| name = "" | ||
|
|
||
| if transaction_style == "url": | ||
| try: |
There was a problem hiding this comment.
Events lose Bottle route names
High Severity
Removing _set_transaction_name_and_source from the request event processor means error and message events no longer get the Bottle route name. _set_segment_name_and_source only runs after the handler returns, so events captured during the request keep the generic WSGI name instead of the matched route.
Reviewed by Cursor Bugbot for commit 0633083. Configure here.
Codecov Results 📊✅ 110960 passed | ⏭️ 5428 skipped | Total: 116388 | Pass Rate: 95.34% | Execution Time: 384m 20s 📊 Comparison with Base Branch
➖ Removed Tests (2)View removed tests
All tests are passing successfully. ✅ Patch coverage is 100.00%. Project has 2250 uncovered lines. Coverage diff@@ Coverage Diff @@
## main #PR +/-##
==========================================
+ Coverage 90.24% 90.57% +0.33%
==========================================
Files 193 185 -8
Lines 25480 23864 -1616
Branches 9382 8798 -584
==========================================
+ Hits 22994 21614 -1380
- Misses 2486 2250 -236
- Partials 1431 1366 -65Generated by Codecov Action |


This should not be needed anymore.