Make scrubber initialization more explicit

Author: antonpirker

sentry_sdk/client.py

@@ -128,7 +128,11 @@ def _get_options(*args, **kwargs):
         rv["traces_sample_rate"] = 1.0
 
     if rv["event_scrubber"] is None:
-        rv["event_scrubber"] = EventScrubber(send_default_pii=rv["send_default_pii"])
+        rv["event_scrubber"] = EventScrubber(
+            send_default_pii=(
+                False if rv["send_default_pii"] is None else rv["send_default_pii"]
+            )
+        )
 
     if rv["socket_options"] and not isinstance(rv["socket_options"], list):
         logger.warning(

tests/test_scope.py

@@ -811,7 +811,7 @@ def test_should_send_default_pii_false(sentry_init):
     assert should_send_default_pii() is False
 
 
-def test_should_send_default_pii_default_fals(sentry_init):
+def test_should_send_default_pii_default_false(sentry_init):
     sentry_init()
 
     assert should_send_default_pii() is False