Skip to content

fix: Suppress warning about instantiated SentryLogger default value#505

Merged
limbonaut merged 2 commits intomainfrom
fix/sentrylogger-default-value-warning
Feb 11, 2026
Merged

fix: Suppress warning about instantiated SentryLogger default value#505
limbonaut merged 2 commits intomainfrom
fix/sentrylogger-default-value-warning

Conversation

@limbonaut
Copy link
Collaborator

@limbonaut limbonaut commented Feb 10, 2026

Add PROPERTY_USAGE_NONE to the logger property binding on SentrySDK so Godot doesn't treat the constructor-instantiated SentryLogger as a serializable default value, which triggers a warning on startup.

The property was missing PROPERTY_USAGE_NONE, making it serializable when it shouldn't be. The same pattern is already used for logger_limits and experimental in SentryOptions.

Add PROPERTY_USAGE_NONE to the logger property binding to prevent
Godot from treating the constructor-instantiated SentryLogger as a
serializable default value, which triggers a harmless but annoying
warning on startup.

Fixes #437
Co-Authored-By: Claude <noreply@anthropic.com>
@github-actions
Copy link
Contributor

github-actions bot commented Feb 10, 2026

Semver Impact of This PR

🟢 Patch (bug fixes)

📋 Changelog Preview

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


Bug Fixes 🐛

  • Suppress warning about instantiated SentryLogger default value by limbonaut in #505
  • Recreate SentryOptions on each init cycle by limbonaut in #503
  • Suppress warning about instantiated SentryLogger default value by limbonaut in #505

Internal Changes 🔧

  • (build) Produce iOS xcframework with framework bundle slices by limbonaut in #500
  • (project) Add isolated test runner in tools panel by limbonaut in #491
  • (release) Fix changelog-preview permissions by BYK in #486
  • Enable logs and set screenshot level to info in test project by limbonaut in #506
  • Ship only addons/sentry/ in build artifacts and release zip by limbonaut in #504
  • Use unique entry point by limbonaut in #499
  • Fix usage of non-existant OS.get_memory_info() key by limbonaut in #493
  • Use int64 for context values on native by limbonaut in #492
  • Extract hash funcs and decouple implementation from STL by limbonaut in #490
  • Add more tools in test project by limbonaut in #489
  • Align enable_logs option with other SDKs by limbonaut in #484

Other

  • release: 1.3.2 by limbonaut in 527dcfc5
  • test: iOS end-to-end in CI by limbonaut in #462

🤖 This preview updates automatically when you update the PR.

Co-Authored-By: Claude <noreply@anthropic.com>
@limbonaut limbonaut marked this pull request as ready for review February 10, 2026 21:09
@limbonaut limbonaut merged commit 98157fc into main Feb 11, 2026
57 checks passed
@limbonaut limbonaut deleted the fix/sentrylogger-default-value-warning branch February 11, 2026 08:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Warning about instantiated SentryLogger used as default value

3 participants