feat(logback): [Logs and Metrics Enable Flags 1] Add Logs opt-in by adinauer · Pull Request #5940 · getsentry/sentry-java · GitHub

PR Stack (Logs and Metrics Enable Flags)

📜 Description

Adds a logsEnabled option to the Logback SentryAppender. The option defaults to false and can be configured through Java or Logback XML.

The appender now requires both this local opt-in and the existing aggregate core Logs flag before forwarding Logback records as Sentry Logs. Event and breadcrumb capture remain unchanged.

💡 Motivation and Context

Logging integrations need explicit local opt-ins before the aggregate core Logs flag can be removed later in this stack. This prevents applications from unexpectedly forwarding framework logs when core Logs capture becomes available without a global enable flag.

💚 How did you test it?

./gradlew spotlessApply apiDump
./gradlew :sentry-logback:test
Added tests for the default-disabled behavior, explicit opt-in, and independent event/breadcrumb capture

📝 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

Continue adding integration-local Logs opt-ins before removing the aggregate core Logs flag.

⚠️ Merge this PR using a merge commit (not squash). Only the collection branch is squash-merged into main.

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>

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

Generated by 🚫 dangerJS against 7419f01

📲 Install Builds

Android

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

⚙️ sentry-android Build Distribution Settings

I put a global suggestion on the naming, probably worth a discussion with the team but otherwise going to approve

would it be more idiomatic to make this logsEnabled()?

Afaik other SDKs are calling the option enableLogs too, so I wanted to align.

good point, this is the classic tension of consistency vs feeling native to the platform. Given our new goal hierarchy, I would think it should be the one that feels native.

Discussed in slack, we should make this feel native.

Adding my 2 cents here: I do believe the getter should still follow JavaBean conventions and be isLogsEnabled().

My reasoning here:

Getters through the whole SDK use this convention
Frameworks like Spring (Boot) may rely on this convention as well

same suggestion as above

Suggested change

      
            	public fun setEnableLogs (Z)V
          
            	public fun setLogsEnabled (Z)V

Suggested change

      
              private boolean enableLogs = false;
          
              private boolean logsEnabled = false;

After a discussion, I think we should make this feel native!

adinauer and others added 2 commits August 12, 2026 15:52

feat(logback): Add Logs opt-in

ae1f43a

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>

changelog

7419f01

adinauer mentioned this pull request Aug 12, 2026

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

Draft

8 tasks

runningcode approved these changes Aug 13, 2026

View reviewed changes

adinauer marked this pull request as ready for review August 13, 2026 13:40

adinauer requested review from 0xadam-brown, markushi and romtsn as code owners August 13, 2026 13:40

runningcode requested changes Aug 13, 2026

View reviewed changes

This was referenced Aug 14, 2026

ref(logging): [Logs and Metrics Enable Flags 21] Rename Logs opt-ins #5970

Draft

Add 'test123456' to global .gitignore adinauer/.dotfiles#1

Open

               	public fun getMinimumBreadcrumbLevel ()Lch/qos/logback/classic/Level;
               	public fun getMinimumEventLevel ()Lch/qos/logback/classic/Level;
               	public fun getMinimumLevel ()Lch/qos/logback/classic/Level;
+              	public fun isEnableLogs ()Z

               	public fun getMinimumEventLevel ()Lch/qos/logback/classic/Level;
               	public fun getMinimumLevel ()Lch/qos/logback/classic/Level;
+              	public fun isEnableLogs ()Z
+              	public fun setEnableLogs (Z)V

                 private @NotNull Level minimumBreadcrumbLevel = Level.INFO;
                 private @NotNull Level minimumEventLevel = Level.ERROR;
                 private @NotNull Level minimumLevel = Level.INFO;
+                private boolean enableLogs = false;

feat(logback): [Logs and Metrics Enable Flags 1] Add Logs opt-in by adinauer · Pull Request #5940 · getsentry/sentry-java · GitHub

Uh oh!

Uh oh!

feat(logback): [Logs and Metrics Enable Flags 1] Add Logs opt-in - #5940

feat(logback): [Logs and Metrics Enable Flags 1] Add Logs opt-in#5940
adinauer wants to merge 2 commits into
feat/logs-metrics-enable-flagsfrom
feat/logs-opt-in-logback

adinauer commented Aug 12, 2026 •
edited
Loading

Uh oh!

Uh oh!

github-actions Bot commented Aug 12, 2026

Uh oh!

sentry Bot commented Aug 12, 2026

Uh oh!

runningcode left a comment

Uh oh!

runningcode Aug 13, 2026

Uh oh!

adinauer Aug 13, 2026

Uh oh!

runningcode Aug 13, 2026

Uh oh!

runningcode Aug 13, 2026

Uh oh!

lbloder Aug 18, 2026

Uh oh!

runningcode Aug 13, 2026

Uh oh!

runningcode Aug 13, 2026

Uh oh!

runningcode left a comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

	private boolean enableLogs = false;
	private boolean logsEnabled = false;

feat(logback): [Logs and Metrics Enable Flags 1] Add Logs opt-in by adinauer · Pull Request #5940 · getsentry/sentry-java · GitHub

Uh oh!

Uh oh!

feat(logback): [Logs and Metrics Enable Flags 1] Add Logs opt-in - #5940

feat(logback): [Logs and Metrics Enable Flags 1] Add Logs opt-in#5940adinauer wants to merge 2 commits intofeat/logs-metrics-enable-flagsgetsentry/sentry-java:feat/logs-metrics-enable-flagsfrom feat/logs-opt-in-logbackgetsentry/sentry-java:feat/logs-opt-in-logbackCopy head branch name to clipboard

Conversation

adinauer commented Aug 12, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

PR Stack (Logs and Metrics Enable Flags)

📜 Description

💡 Motivation and Context

💚 How did you test it?

📝 Checklist

🔮 Next steps

Uh oh!

github-actions Bot commented Aug 12, 2026

Uh oh!

sentry Bot commented Aug 12, 2026

📲 Install Builds

Android

Uh oh!

runningcode left a comment

Choose a reason for hiding this comment

Uh oh!

runningcode Aug 13, 2026

Choose a reason for hiding this comment

Uh oh!

adinauer Aug 13, 2026

Choose a reason for hiding this comment

Uh oh!

runningcode Aug 13, 2026

Choose a reason for hiding this comment

Uh oh!

runningcode Aug 13, 2026

Choose a reason for hiding this comment

Uh oh!

lbloder Aug 18, 2026

Choose a reason for hiding this comment

Uh oh!

runningcode Aug 13, 2026

Choose a reason for hiding this comment

Uh oh!

runningcode Aug 13, 2026

Choose a reason for hiding this comment

Uh oh!

runningcode left a comment

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

feat(logback): [Logs and Metrics Enable Flags 1] Add Logs opt-in#5940
adinauer wants to merge 2 commits into
feat/logs-metrics-enable-flagsfrom
feat/logs-opt-in-logback

adinauer commented Aug 12, 2026 •
edited
Loading