Skip to content

fix(core)!: Remove enableLogs option - #23319

Merged
nicohrubec merged 1 commit into
developfrom
feat/remove-enable-logs-flag
Aug 12, 2026
Merged

fix(core)!: Remove enableLogs option#23319
nicohrubec merged 1 commit into
developfrom
feat/remove-enable-logs-flag

Conversation

@nicohrubec

@nicohrubec nicohrubec commented Aug 12, 2026

Copy link
Copy Markdown
Member

Removes the enableLogs option for v11. Logs are now captured whenever a logging API (Sentry.logger.*) or a logging integration is used, with no way to disable them.

Fixes #23308

Similar to #23321

Removes the enableLogs option entirely. Logs are now captured whenever a
logging API or logging integration is used, with no way to disable them.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown
Contributor

size-limit report 📦

⚠️ Warning: Base artifact is not the latest one, because the latest workflow run is not done yet. This may lead to incorrect results. Try to re-run all tests to get up to date results.

Path Size % Change Change
@sentry/browser 30.28 kB -0.06% -18 B 🔽
@sentry/browser - with treeshaking flags 28.44 kB -0.1% -26 B 🔽
@sentry/browser - with treeshaking flags tracing without tracing 26.79 kB -0.09% -24 B 🔽
@sentry/browser (incl. Tracing) 48.51 kB -0.04% -19 B 🔽
@sentry/browser (incl. Tracing + Span Streaming) 48.53 kB -0.04% -18 B 🔽
@sentry/browser (incl. Tracing, Profiling) 51.41 kB -0.04% -19 B 🔽
@sentry/browser (incl. Tracing, Replay) 87.9 kB -0.03% -25 B 🔽
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags 77.32 kB -0.04% -26 B 🔽
@sentry/browser (incl. Tracing, Replay with Canvas) 92.62 kB -0.03% -22 B 🔽
@sentry/browser (incl. Tracing, Replay, Feedback) 105.32 kB -0.03% -26 B 🔽
@sentry/browser (incl. Feedback) 47.59 kB -0.07% -33 B 🔽
@sentry/browser (incl. sendFeedback) 35.1 kB -0.05% -16 B 🔽
@sentry/browser (incl. FeedbackAsync) 40.24 kB -0.06% -22 B 🔽
@sentry/browser (incl. Metrics) 31.26 kB -0.07% -21 B 🔽
@sentry/browser (incl. Logs) 31.49 kB -0.16% -49 B 🔽
@sentry/browser (incl. Metrics & Logs) 32.15 kB -0.19% -58 B 🔽
@sentry/react 32.05 kB -0.11% -34 B 🔽
@sentry/react (incl. Tracing) 50.69 kB -0.06% -29 B 🔽
@sentry/vue 35.28 kB -0.07% -22 B 🔽
@sentry/vue (incl. Tracing) 50.46 kB -0.05% -22 B 🔽
@sentry/svelte 30.29 kB -0.07% -21 B 🔽
CDN Bundle 31.58 kB -0.07% -19 B 🔽
CDN Bundle (incl. Tracing) 48.83 kB -0.04% -17 B 🔽
CDN Bundle (incl. Logs, Metrics) 33.78 kB -0.11% -35 B 🔽
CDN Bundle (incl. Tracing, Logs, Metrics) 50.76 kB -0.1% -47 B 🔽
CDN Bundle (incl. Replay, Logs, Metrics) 74.29 kB -0.08% -57 B 🔽
CDN Bundle (incl. Tracing, Replay) 86.42 kB -0.03% -21 B 🔽
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) 88.27 kB -0.06% -47 B 🔽
CDN Bundle (incl. Tracing, Replay, Feedback) 92.13 kB -0.03% -21 B 🔽
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) 94.08 kB -0.05% -47 B 🔽
CDN Bundle - uncompressed 93.77 kB -0.08% -66 B 🔽
CDN Bundle (incl. Tracing) - uncompressed 146.59 kB -0.05% -68 B 🔽
CDN Bundle (incl. Logs, Metrics) - uncompressed 100.1 kB -0.14% -132 B 🔽
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed 152.32 kB -0.09% -134 B 🔽
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed 229.04 kB -0.06% -134 B 🔽
CDN Bundle (incl. Tracing, Replay) - uncompressed 265.85 kB -0.03% -68 B 🔽
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed 271.56 kB -0.05% -134 B 🔽
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed 279.55 kB -0.03% -68 B 🔽
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed 285.25 kB -0.05% -134 B 🔽
@sentry/nextjs (client) 53.24 kB -0.07% -32 B 🔽
@sentry/sveltekit (client) 48.92 kB -0.05% -24 B 🔽
@sentry/core/server 65.34 kB -0.12% -73 B 🔽
@sentry/core/browser 51.69 kB -0.15% -77 B 🔽
@sentry/node 117.92 kB -0.03% -35 B 🔽
@sentry/node/import (ESM hook with diagnostics-channel injection) 0 B added added
@sentry/node - without tracing 82.03 kB -0.04% -30 B 🔽
@sentry/aws-serverless 91.42 kB -0.04% -36 B 🔽
@sentry/cloudflare (withSentry) - minified 213.8 kB -0.04% -80 B 🔽
@sentry/cloudflare (withSentry) 528.18 kB -0.03% -119 B 🔽

View base workflow run

@nicohrubec
nicohrubec marked this pull request as ready for review August 12, 2026 08:54
@nicohrubec
nicohrubec requested review from a team as code owners August 12, 2026 08:54
@nicohrubec
nicohrubec requested review from JPeer264, logaretm and mydea and removed request for a team August 12, 2026 08:54

@JPeer264 JPeer264 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Fine by me, but will we do the same for metrics? If not it seems kinda odd to not have the option

Copy link
Copy Markdown
Member Author

yes we'll do the same for metrics #23321

@nicohrubec nicohrubec changed the title feat(core)!: Remove enableLogs option fix(core)!: Remove enableLogs option Aug 12, 2026
@nicohrubec
nicohrubec merged commit beb0f34 into develop Aug 12, 2026
532 of 538 checks passed
@nicohrubec
nicohrubec deleted the feat/remove-enable-logs-flag branch August 12, 2026 14:27
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.

Remove enableLogs flag from the SDK

3 participants