Skip to content

Commit ec74f25

Browse files
committed
chore(sdk): Use add_full_stack by default
The option has been enabled since the weekend (getsentry/sentry-options-automator#3902). This undoes some of what was added [here](#91062).
1 parent cb103d0 commit ec74f25

File tree

3 files changed

+2
-9
lines changed

3 files changed

+2
-9
lines changed

src/sentry/conf/types/sdk_config.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,6 @@ class SdkConfig(TypedDict):
1616
auto_enabling_integrations: bool
1717
keep_alive: NotRequired[bool]
1818
spotlight: NotRequired[str | bool | None]
19-
add_full_stack: NotRequired[bool]
2019
send_client_reports: NotRequired[bool]
2120
traces_sampler: NotRequired[Callable[[dict[str, Any]], float]]
2221
before_send: NotRequired[Callable[[Event, Hint], Event | None]]

src/sentry/options/defaults.py

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -2367,13 +2367,6 @@
23672367

23682368
# END: SDK Crash Detection
23692369

2370-
# Whether to add the full stack trace to Python errors.
2371-
register(
2372-
"sentry_sdk.add_full_stack",
2373-
default=False,
2374-
flags=FLAG_AUTOMATOR_MODIFIABLE,
2375-
)
2376-
23772370
register(
23782371
# Lists the shared resource ids we want to account usage for.
23792372
"shared_resources_accounting_enabled",

src/sentry/utils/sdk.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -296,7 +296,8 @@ def _get_sdk_options() -> tuple[SdkConfig, Dsns]:
296296
before_send_log=before_send_log,
297297
enable_logs=True,
298298
)
299-
sdk_options["add_full_stack"] = options.get("sentry_sdk.add_full_stack", False)
299+
# Captured errors will include the full stack trace.
300+
sdk_options["add_full_stack"] = True
300301

301302
# Modify SENTRY_SDK_CONFIG in your deployment scripts to specify your desired DSN
302303
dsns = Dsns(

0 commit comments

Comments
 (0)