Skip to content

feat(core): Remove Logs and Metrics enable flags - #5939

Merged
adinauer merged 166 commits into
9.x.xfrom
feat/logs-metrics-enable-flags
Oct 13, 2026
Merged

adinauer merged 166 commits into
9.x.xfrom
feat/logs-metrics-enable-flags

Conversation

@adinauer

@adinauer adinauer commented Aug 12, 2026 •

Copy link
Copy Markdown
Member

PR Stack (Logs and Metrics Enable Flags)


📜 Description

Collection PR for the Logs and Metrics enable-flag removal stack. Squash-merge this once all stack PRs are merged.

💡 Motivation and Context

Removes the aggregate Logs and Metrics enable flags, adds integration-local Logs opt-ins, and avoids starting unused batch worker threads.

💚 How did you test it?

Each PR in the stack includes focused tests. The completed stack will run the full repository checks and aggregate Javadocs.

📝 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

Merge each stack PR into this collection branch in order using merge commits, then squash-merge this PR into 9.x.x.

@github-actions

github-actions Bot commented Aug 12, 2026 •

Copy link
Copy Markdown
Contributor
Messages
📖 Do not forget to update Sentry-docs with your feature once the pull request gets approved.

Generated by 🚫 dangerJS against fc98241

@sentry

sentry Bot commented Aug 12, 2026 •

Copy link
Copy Markdown

📲 Install Builds

Android

🔗 App Name App ID Version Configuration
SDK Size io.sentry.tests.size 8.59.0 (1) release

⚙️ sentry-android Build Distribution Settings

Require an explicit Logback appender opt-in before forwarding records as Sentry Logs. Keep event and breadcrumb capture independent while the aggregate core Logs flag remains in place.

Co-Authored-By: Claude <noreply@anthropic.com>
adinauer and others added 2 commits August 12, 2026 15:54
Require an explicit Log4j2 appender opt-in before forwarding records as Sentry Logs. Preserve existing constructors and factory methods with Logs disabled by default while keeping events and breadcrumbs unchanged.

Co-Authored-By: Claude <noreply@anthropic.com>
adinauer and others added 2 commits August 12, 2026 16:02
Require an explicit JUL handler opt-in before forwarding records as Sentry Logs. Support Java and LogManager configuration while keeping event and breadcrumb capture unchanged.

Co-Authored-By: Claude <noreply@anthropic.com>
adinauer and others added 2 commits August 12, 2026 16:07
Require an explicit Timber-local opt-in before forwarding Timber messages as Sentry Logs. Add Android options and manifest configuration for auto-installed integrations while preserving event and breadcrumb capture.

Co-Authored-By: Claude <noreply@anthropic.com>
adinauer and others added 2 commits August 12, 2026 16:21
Require an explicit Android option before forwarding Logcat calls as Sentry Logs. Support Java and manifest configuration while keeping breadcrumbs and Android Log calls unchanged.

Co-Authored-By: Claude <noreply@anthropic.com>
adinauer and others added 2 commits August 12, 2026 16:29
Add a Spring Boot logging property that controls Sentry Logs forwarding from auto-configured Logback appenders. Keep appender installation, events, and breadcrumbs independent from the new opt-in.

Co-Authored-By: Claude <noreply@anthropic.com>
adinauer and others added 2 commits August 12, 2026 16:35
Capture manual Sentry Logs without an aggregate enable option and always create the configured logger batch processor. Keep automatic logging integrations controlled by their local opt-ins and remove legacy Logs configuration from active samples and fixtures.

Co-Authored-By: Claude <noreply@anthropic.com>
adinauer and others added 2 commits August 12, 2026 16:44
Detect explicit io.sentry.logs.enabled Android manifest metadata and emit tailored migration warnings without changing the integration-local Logs options.

Co-Authored-By: Claude <noreply@anthropic.com>
Keep parsing the nullable logs.enabled external option and emit tailored migration warnings for explicit true and false values without changing manual Logs capture.

Co-Authored-By: Claude <noreply@anthropic.com>
Inspect the Spring Environment before SDK initialization and emit tailored migration warnings for explicit sentry.logs.enabled values without binding or applying the obsolete property.

Co-Authored-By: Claude <noreply@anthropic.com>
adinauer and others added 19 commits October 9, 2026 11:36
Propagate removal of unused no-op telemetry batch processors through the
stack while preserving branch ancestry.
Propagate removal of unused no-op telemetry batch processors through the
stack while preserving branch ancestry.
Propagate removal of unused no-op telemetry batch processors through the
stack while preserving branch ancestry.
Propagate removal of unused no-op telemetry batch processors through the
stack while preserving branch ancestry.
Propagate removal of unused no-op telemetry batch processors through the
stack while preserving branch ancestry.
feat(core): [Logs and Metrics Enable Flags 7] Remove Logs enable flag
perf(core): [Logs and Metrics Enable Flags 8] Avoid unused Logs worker thread
feat(core): [Logs and Metrics Enable Flags 9] Remove Metrics enable flag
…st-use

perf(core): [Logs and Metrics Enable Flags 10] Avoid unused Metrics worker thread
fix(jul): [Logs and Metrics Enable Flags 11] Preserve null-message records
docs: [Logs and Metrics Enable Flags 12] Update options guidance
fix(android): [Logs and Metrics Enable Flags 13] Read Timber option lazily
Remove the constructor superseded by the minimumLevel overload in 8.16.0
at the v9 breaking-change boundary.

Co-Authored-By: Claude <noreply@anthropic.com>
Rename the Timber and Logcat manifest options to describe Sentry's log
capture direction consistently with their Android option accessors.

Co-Authored-By: Claude <noreply@anthropic.com>
ref(logging): [Logs and Metrics Enable Flags 14] Rename Logs opt-ins
Resolve conflicts while preserving source-local Logs controls and the removal of aggregate Logs and Metrics flags.

Co-Authored-By: Claude <noreply@anthropic.com>
List the exact option, property, attribute, and manifest names for every logging integration so users can enable the intended source after upgrading.

Co-Authored-By: Claude <noreply@anthropic.com>
Lead with removal of the aggregate Logs and Metrics flags, list every source-local Logs opt-in, and keep constructor migrations as secondary details.

Co-Authored-By: Claude <noreply@anthropic.com>
List the Spring Boot logging property first among the source-local Logs controls.

Co-Authored-By: Claude <noreply@anthropic.com>
@adinauer
adinauer marked this pull request as ready for review October 13, 2026 07:21
Reference the collection PR from the consolidated breaking-change entry.

Co-Authored-By: Claude <noreply@anthropic.com>

@lbloder lbloder left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍
minor nits on the changelog commented

Comment thread CHANGELOG.md Outdated
Comment thread CHANGELOG.md Outdated
Comment thread CHANGELOG.md Outdated
List every configuration surface affected by the aggregate Logs and
Metrics option removal and structure the migration guidance clearly.

Co-Authored-By: Claude <noreply@anthropic.com>
@adinauer
adinauer merged commit 48e97fb into 9.x.x Oct 13, 2026
64 of 66 checks passed
@adinauer
adinauer deleted the feat/logs-metrics-enable-flags branch October 13, 2026 13:53
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.

2 participants