Skip to content

No longer log a warning if a logging integration cannot initialize Sentry due to missing DSN.#5075

Open
adinauer wants to merge 1 commit intomainfrom
fix/silence-init-failed-warning-for-logger-integrations
Open

No longer log a warning if a logging integration cannot initialize Sentry due to missing DSN.#5075
adinauer wants to merge 1 commit intomainfrom
fix/silence-init-failed-warning-for-logger-integrations

Conversation

@adinauer
Copy link
Member

@adinauer adinauer commented Feb 5, 2026

📜 Description

No longer log a warning if a logging integration cannot initialize Sentry due to missing DSN.

💡 Motivation and Context

Fixes #4531

This warning is causing lots of confusion, especially for customers, who have the logging integration auto installed by one of the build plugins.

💚 How did you test it?

📝 Checklist

  • I added GH Issue ID & Linear ID
  • I added tests to verify the changes.
  • No new PII added or SDK only sends newly added PII if sendDefaultPII is enabled.
  • I updated the docs if needed.
  • I updated the wizard if needed.
  • Review from the native team if needed.
  • No breaking change or entry added to the changelog.
  • No breaking change for hybrid SDKs or communicated to hybrid SDKs.

🔮 Next steps

@github-actions
Copy link
Contributor

github-actions bot commented Feb 5, 2026

Semver Impact of This PR

None (no version bump detected)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


New Features ✨

  • (distribution) Add install_groups support by runningcode in #5062
  • (spotlight) Extract SpotlightIntegration to separate module by romtsn in #5064

Bug Fixes 🐛

  • Establish native exception mechanisms by supervacuus in #5052

Internal Changes 🔧

Deps

  • Bump github/codeql-action from 4.31.11 to 4.32.0 by dependabot in #5067
  • Bump gradle/actions from 5.0.0 to 5.0.1 by dependabot in #5068
  • Bump getsentry/craft from 2.20.0 to 2.20.1 by dependabot in #5069
  • Bump urllib3 from 2.6.0 to 2.6.3 in the pip group across 1 directory by dependabot in #5003
  • Update Native SDK to v0.12.4 by github-actions in #5061
  • Bump getsentry/github-workflows/.github/workflows/updater.yml from 2 to 3 by dependabot in #4884
  • Bump actions/cache from 4 to 5 by dependabot in #4997
  • Bump github/codeql-action from 4.31.10 to 4.31.11 by dependabot in #5057
  • Bump getsentry/craft from 2.19.0 to 2.20.0 by dependabot in #5058

Other

  • (android) Update targetSdk to API 36 (Android 16) by markushi in #5016
  • (ci) Write permission for statuses in changelog preview by supervacuus in #5053
  • (samples) Convert main screen to Jetpack Compose by markushi in #5017

Other

  • No longer log a warning if a logging integration cannot initialize Sentry due to missing DSN. by adinauer in #5075

🤖 This preview updates automatically when you update the PR.

@github-actions
Copy link
Contributor

github-actions bot commented Feb 5, 2026

Fails
🚫 Please consider adding a changelog entry for the next release.

Instructions and example for changelog

Please add an entry to CHANGELOG.md to the "Unreleased" section. Make sure the entry includes this PR's number.

Example:

## Unreleased

### Features

- No longer log a warning if a logging integration cannot initialize Sentry due to missing DSN ([#5075](https://github.com/getsentry/sentry-java/pull/5075))

If none of the above apply, you can opt out of this check by adding #skip-changelog to the PR description or adding a skip-changelog label.

Generated by 🚫 dangerJS against d80c4fa

@github-actions
Copy link
Contributor

github-actions bot commented Feb 5, 2026

Performance metrics 🚀

  Plain With Sentry Diff
Startup time 308.92 ms 342.29 ms 33.37 ms
Size 1.58 MiB 2.19 MiB 619.17 KiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
27d7cf8 397.90 ms 498.65 ms 100.75 ms
e59e22a 329.74 ms 383.31 ms 53.57 ms
ae7fed0 293.84 ms 380.22 ms 86.38 ms
b3d8889 371.84 ms 447.49 ms 75.65 ms
5f14e5d 325.76 ms 368.32 ms 42.56 ms
fcec2f2 311.35 ms 384.94 ms 73.59 ms
d15471f 307.28 ms 381.85 ms 74.57 ms
fc5ccaf 279.11 ms 353.34 ms 74.23 ms
8687935 294.00 ms 304.02 ms 10.02 ms
ee747ae 400.46 ms 423.61 ms 23.15 ms

App size

Revision Plain With Sentry Diff
27d7cf8 1.58 MiB 2.12 MiB 549.42 KiB
e59e22a 1.58 MiB 2.20 MiB 635.34 KiB
ae7fed0 1.58 MiB 2.12 MiB 551.77 KiB
b3d8889 1.58 MiB 2.10 MiB 535.07 KiB
5f14e5d 1.58 MiB 2.19 MiB 620.00 KiB
fcec2f2 1.58 MiB 2.12 MiB 551.51 KiB
d15471f 1.58 MiB 2.13 MiB 559.54 KiB
fc5ccaf 1.58 MiB 2.13 MiB 557.54 KiB
8687935 1.58 MiB 2.19 MiB 619.17 KiB
ee747ae 1.58 MiB 2.10 MiB 530.95 KiB

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Failed to init Sentry during appender initialization: DSN is required

2 participants