Skip to content
Snippets Groups Projects

tor-chanmgr: remove panics in debug builds

Merged opara requested to merge opara/arti:remove-debug-panic into main

These were supposed to fail loudly in debug builds by panicking, but panics are mostly useless for debugging in async applications that use a runtime which catches panics. So we'll just log the error instead.

One of these panics actually occurs often in practice (see #1716 (closed) and #1731).

Merge request reports

Pipeline #222926 failed

Pipeline failed for cad2a728 on opara:remove-debug-panic

Approved by

Merged by Alexander Hansen FærøyAlexander Hansen Færøy 3 months ago (Nov 12, 2024 3:04pm UTC)

Merge details

Pipeline #223095 failed

Pipeline failed for 69125b90 on main

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading