Skip to content

Fix sentry logger init failing for edge cases#37673

Merged
peppy merged 1 commit intoppy:masterfrom
peppy:fix-sentry-logger-fail
May 8, 2026
Merged

Fix sentry logger init failing for edge cases#37673
peppy merged 1 commit intoppy:masterfrom
peppy:fix-sentry-logger-fail

Conversation

@peppy
Copy link
Copy Markdown
Member

@peppy peppy commented May 8, 2026

Closes #37650.

@peppy peppy force-pushed the fix-sentry-logger-fail branch from 92e0674 to 69bae47 Compare May 8, 2026 08:20
@pull-request-size pull-request-size Bot added size/XS and removed size/S labels May 8, 2026
@peppy peppy force-pushed the fix-sentry-logger-fail branch from 69bae47 to 5a16725 Compare May 8, 2026 08:21
@peppy peppy added the area:test coverage Adds or modifies test coverage without adding any functionality. label May 8, 2026
@bdach bdach self-requested a review May 8, 2026 08:30
@bdach
Copy link
Copy Markdown
Collaborator

bdach commented May 8, 2026

https://develop.sentry.dev/sdk/foundations/client/#client-lifecycle

SDKs MUST accept an empty DSN as valid configuration. If initialized with an empty or missing DSN, the SDK MUST NOT send any data over the network. Depending on the platform, the SDK MAY reduce its runtime footprint to a minimum.

👍

@peppy peppy merged commit c74c54d into ppy:master May 8, 2026
7 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:test coverage Adds or modifies test coverage without adding any functionality. size/XS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Ruleset template test browser crashing due to invalid sentry dsn

2 participants