fix #26 (comment)

Author: germankay

ckanext/push_errors/plugin.py

@@ -6,7 +6,6 @@
 from ckan.plugins import toolkit
 from ckanext.push_errors.logging import PushErrorHandler, push_message
 from ckanext.push_errors.cli import push_errors as push_errors_commands
-from ckanext.push_errors.error_logger import log_error
 from ckanext.push_errors.utils import get_error_trace_id
 
 from ckanext.push_errors.blueprints.push_errors import push_error_bp
@@ -45,12 +44,6 @@ def error_handler(exception):
                 if isinstance(exception, skip_types_if_anon):
                     return None
 
-            # Registrar usando log_error (deduplicación y silenciado)
-            try:
-                log_error(exception)
-            except Exception as log_err:
-                log.warning(f'push-errors: log_error failed: {log_err}')
-
             exception_str = f'{exception} [({type(exception).__name__})]'
             # get the stacktrace
             trace = traceback.format_exc()

ckanext/push_errors/tests/test_utils.py

@@ -21,8 +21,10 @@ def test_get_error_trace_id_returns_path_and_line():
         raise_custom_error()
     except CustomError as e:
         trace_id = get_error_trace_id(e)
-        assert ":" in trace_id
-        assert not trace_id.endswith(".py")  # it only ends with ":<line>"
+        assert ":" in trace_id, "Trace ID should contain a colon"
+        path, line = trace_id.rsplit(":", 1)
+        assert path.endswith("test_utils.py"), f"Expected path to end with test_utils.py, got {path}"
+        assert line.isdigit(), f"Expected a line number, got {line}"
 
 
 def test_get_error_trace_id_unknown_when_no_tb():