.

Author: sentrivana

sentry_sdk/integrations/ariadne.py

@@ -120,13 +120,13 @@ def inner(event: Event, hint: dict[str, Any]) -> Event:
         if not isinstance(data, dict):
             return event
 
-            if should_send_default_pii() and should_include_request_body(get_client()):
-                request_info = event.setdefault("request", {})
-                request_info["api_target"] = "graphql"
-                request_info["data"] = data
+        if should_send_default_pii() and should_include_request_body(get_client()):
+            request_info = event.setdefault("request", {})
+            request_info["api_target"] = "graphql"
+            request_info["data"] = data
 
-            elif event.get("request", {}).get("data"):
-                del event["request"]["data"]
+        elif event.get("request", {}).get("data"):
+            del event["request"]["data"]
 
         return event
 

tests/integrations/clickhouse_driver/test_clickhouse_driver.py

@@ -677,7 +677,7 @@ def test_clickhouse_dbapi_breadcrumbs_with_pii(sentry_init, capture_events) -> N
                 "server.address": "localhost",
                 "server.port": 9000,
                 "db.params": {"minv": 150},
-                "db.result": [[["370"]], [["'sum(x)'", "'Int64'"]]],
+                "db.result": [[[370]], [["sum(x)", "Int64"]]],
             },
             "message": "SELECT sum(x) FROM test WHERE x > 150",
             "type": "default",

tests/integrations/pure_eval/test_pure_eval.py

@@ -85,4 +85,4 @@ def foo():
             "s",
             "events",
         }
-        assert len(frame_vars) == 99
+        assert len(frame_vars) == 14