Skip to content

Conversation

@tustanivsky
Copy link
Collaborator

This PR adds a new plugin setting that allows disabling the automatic propagation of additional data (e.g., tags and contexts) to Unreal Engine’s Crash Reporter. Disabling this is useful when Crash Reporter is not used/needed as it allows to avoid unnecessary instantiation of the FGenericPlatformSentryCrashContext wrapper object.

Related to:

Closes:

@github-actions
Copy link
Contributor

github-actions bot commented Dec 22, 2025

Fails
🚫 Please consider adding a changelog entry for the next release.
Messages
📖 Do not forget to update Sentry-docs with your feature once the pull request gets approved.

Instructions and example for changelog

Please add an entry to CHANGELOG.md to the "Unreleased" section. Make sure the entry includes this PR's number.

Example:

## Unreleased

### Features

- Add setting allowing to opt-out Crash Reporter wrapper instantiation ([#1178](https://github.com/getsentry/sentry-unreal/pull/1178))

If none of the above apply, you can opt out of this check by adding #skip-changelog to the PR description or adding a skip-changelog label.

Generated by 🚫 dangerJS against c365f3a

Copy link
Collaborator

@limbonaut limbonaut left a comment

Choose a reason for hiding this comment

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

Can it be auto-detected if Unreal crash reporter is used instead of adding a setting?

@tustanivsky tustanivsky merged commit f26754d into main Jan 6, 2026
10 of 11 checks passed
@tustanivsky tustanivsky deleted the feat/crash-reporter-wrapper-opt branch January 6, 2026 10:12
@tustanivsky
Copy link
Collaborator Author

Can it be auto-detected if Unreal crash reporter is used instead of adding a setting?

The suggested setting only gives the user control over whether a corresponding wrapper object is created for custom context propagation. The Crash Reporter may still be used even when this setting is disabled (e.g. in older Unreal Engine versions where crashpad integration isn't supported) so auto-detection may not always work as expected.

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.

3 participants