pacnpal/thrillwiki_django_no_react
1
1
Fork 0
mirror of https://github.com/pacnpal/thrillwiki_django_no_react.git synced 2026-02-05 04:05:25 -05:00
Code Issues Packages Projects Releases Wiki Activity

feat: replace direct logger.error with capture_and_log for critical state machine callback failures.

Browse Source
This commit is contained in:
pacnpal
2026-01-04 18:39:58 -05:00
parent 89d9e945b9
commit dd2d09b1c7
1 changed files with 13 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

18
backend/apps/core/state_machine/callback_base.py
View File

@@ -75,6 +75,8 @@ from typing import Any, Optional
from django.db import models from django.db import models
from apps.core.utils import capture_and_log
logger = logging.getLogger(__name__) logger = logging.getLogger(__name__)
@@ -496,9 +498,12 @@ class TransitionCallbackRegistry:
failures.append((callback, None)) failures.append((callback, None))
overall_success = False overall_success = False
if not callback.continue_on_error: if not callback.continue_on_error:
logger.error( capture_and_log(
f"Aborting callback chain - {callback.name} failed " f"and continue_on_error=False" RuntimeError(f'Callback {callback.name} returned False'),
f'Aborting callback chain - continue_on_error=False',
source='state_machine',
severity='medium',
) )
break break
@@ -508,8 +513,11 @@ class TransitionCallbackRegistry:
overall_success = False overall_success = False
if not callback.continue_on_error: if not callback.continue_on_error:
logger.error( capture_and_log(
f"Aborting callback chain - {callback.name} raised exception " f"and continue_on_error=False" e,
f'Aborting callback chain - {callback.name} raised exception',
source='state_machine',
severity='high',
) )
break break