Skip to content

feat!: enable logs by default#1306

Merged
giortzisg merged 3 commits into
masterfrom
feat/enable-logs-by-default
May 22, 2026
Merged

feat!: enable logs by default#1306
giortzisg merged 3 commits into
masterfrom
feat/enable-logs-by-default

Conversation

@giortzisg
Copy link
Copy Markdown
Contributor

@giortzisg giortzisg commented May 19, 2026

Description

This PR enables logs by default to skip double allow behavior. Users should just use sentry.NewLogger or any integration to enable logs. This also shifts EnableLogs to DisableLogs for a global override switch to disable all logging.

Issues

Changelog Entry Instructions

To add a custom changelog entry, uncomment the section above. Supports:

  • Single entry: just write text
  • Multiple entries: use bullet points
  • Nested bullets: indent 4+ spaces

For more details: custom changelog entries

Reminders

Changelog Entry

  • Enable logs by default to skip double allow behavior. Enabling logs now happens once when setting up either sentry.NewLogger or any supported integration. Also the EnableLogs flag changes to DisableLogs for a global override switch

@giortzisg giortzisg requested a review from sl0thentr0py May 19, 2026 12:38
@giortzisg giortzisg self-assigned this May 19, 2026
@linear-code
Copy link
Copy Markdown

linear-code Bot commented May 19, 2026

GO-143

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 21, 2026

Semver Impact of This PR

🔴 Major (breaking changes)

📋 Changelog Preview

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


Breaking Changes 🛠

  • Enable logs by default to skip double allow behavior. Enabling logs now happens once when setting up either sentry.NewLogger or any supported integration. Also the EnableLogs flag changes to DisableLogs for a global override switch by giortzisg in #1306
  • Enable logs by default to skip double allow behavior. Enabling logs now happens once when setting up either sentry.NewLogger or any supported integration. Also the EnableLogs flag changes to DisableLogs for a global override switch by giortzisg in #1306
  • Remove the ContextifyFrames integration. The recommended way to add source context is SCM by giortzisg in #1302

Internal Changes 🔧

Deps

  • Bump github.com/gofiber/fiber/v2 from 2.52.12 to 2.52.13 in /fiber by dependabot in #1300
  • Bump github.com/gofiber/fiber/v2 from 2.52.12 to 2.52.13 in /crosstest by dependabot in #1301
  • Bump getsentry/craft from 2.25.2 to 2.26.2 by dependabot in #1293
  • Bump getsentry/craft/.github/workflows/changelog-preview.yml from 2.25.2 to 2.26.2 by dependabot in #1294

🤖 This preview updates automatically when you update the PR.

@giortzisg giortzisg merged commit ecc95d7 into master May 22, 2026
22 checks passed
@giortzisg giortzisg deleted the feat/enable-logs-by-default branch May 22, 2026 08:58
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.

Enable logging by default [Go]

2 participants