Make it more resilient againt misuse

Author: antonpirker

sentry_sdk/consts.py

@@ -6,6 +6,9 @@
 # up top to prevent circular import due to integration import
 DEFAULT_MAX_VALUE_LENGTH = 1024
 
+DEFAULT_MAX_STACK_FRAMES = 100
+DEFAULT_ADD_FULL_STACK = False
+
 
 # Also needs to be at the top to prevent circular import
 class EndpointType(Enum):
@@ -550,8 +553,8 @@ def __init__(
         cert_file=None,  # type: Optional[str]
         key_file=None,  # type: Optional[str]
         custom_repr=None,  # type: Optional[Callable[..., Optional[str]]]
-        add_full_stack=False,  # type: bool
-        max_stack_frames=50,  # type: Optional[int]
+        add_full_stack=DEFAULT_ADD_FULL_STACK,  # type: bool
+        max_stack_frames=DEFAULT_MAX_STACK_FRAMES,  # type: Optional[int]
     ):
         # type: (...) -> None
         pass

sentry_sdk/utils.py

@@ -26,7 +26,12 @@
 
 import sentry_sdk
 from sentry_sdk._compat import PY37
-from sentry_sdk.consts import DEFAULT_MAX_VALUE_LENGTH, EndpointType
+from sentry_sdk.consts import (
+    DEFAULT_ADD_FULL_STACK,
+    DEFAULT_MAX_STACK_FRAMES,
+    DEFAULT_MAX_VALUE_LENGTH,
+    EndpointType,
+)
 
 from typing import TYPE_CHECKING
 
@@ -837,7 +842,9 @@ def single_exception_from_error_tuple(
 
             # Limit the number of frames
             max_stack_frames = (
-                client_options.get("max_stack_frames") if client_options else None
+                client_options.get("max_stack_frames", DEFAULT_MAX_STACK_FRAMES)
+                if client_options
+                else None
             )
             if max_stack_frames is not None:
                 new_frames = new_frames[:max_stack_frames]
@@ -1179,7 +1186,7 @@ def event_from_exception(
     exc_info = exc_info_from_error(exc_info)
     hint = event_hint_with_exc_info(exc_info)
 
-    if client_options and client_options["add_full_stack"]:
+    if client_options and client_options.get("add_full_stack", DEFAULT_ADD_FULL_STACK):
         full_stack = get_full_stack()
     else:
         full_stack = None