Replace Elastic APM with Sentry APM (#183)

passbook/lib/expression/evaluator.py

@@ -4,8 +4,9 @@ from textwrap import indent
 from typing import Any, Dict, Iterable, Optional
 
 from django.core.exceptions import ValidationError
-from elasticapm import capture_span
 from requests import Session
+from sentry_sdk import start_span
+from sentry_sdk.tracing import Span
 from structlog import get_logger
 
 from passbook.core.models import User
@@ -71,27 +72,31 @@ class BaseEvaluator:
         full_expression += f"\nresult = handler({handler_signature})"
         return full_expression
 
-    @capture_span(name="BaseEvaluator", span_type="lib.evaluator.evaluate")
     def evaluate(self, expression_source: str) -> Any:
         """Parse and evaluate expression. If the syntax is incorrect, a SyntaxError is raised.
         If any exception is raised during execution, it is raised.
         The result is returned without any type-checking."""
-        param_keys = self._context.keys()
-        ast_obj = compile(
-            self.wrap_expression(expression_source, param_keys), self._filename, "exec",
-        )
-        try:
-            _locals = self._context
-            # Yes this is an exec, yes it is potentially bad. Since we limit what variables are
-            # available here, and these policies can only be edited by admins, this is a risk
-            # we're willing to take.
-            # pylint: disable=exec-used
-            exec(ast_obj, self._globals, _locals)  # nosec # noqa
-            result = _locals["result"]
-        except Exception as exc:
-            LOGGER.warning("Expression error", exc=exc)
-            raise
-        return result
+        with start_span(op="lib.evaluator.evaluate") as span:
+            span: Span
+            span.set_data("expression", expression_source)
+            param_keys = self._context.keys()
+            ast_obj = compile(
+                self.wrap_expression(expression_source, param_keys),
+                self._filename,
+                "exec",
+            )
+            try:
+                _locals = self._context
+                # Yes this is an exec, yes it is potentially bad. Since we limit what variables are
+                # available here, and these policies can only be edited by admins, this is a risk
+                # we're willing to take.
+                # pylint: disable=exec-used
+                exec(ast_obj, self._globals, _locals)  # nosec # noqa
+                result = _locals["result"]
+            except Exception as exc:
+                LOGGER.warning("Expression error", exc=exc)
+                raise
+            return result
 
     def validate(self, expression: str) -> bool:
         """Validate expression's syntax, raise ValidationError if Syntax is invalid"""