chore(deps): update Native SDK to v0.16.4 - #2820
Open
github-actions[bot] wants to merge 1 commit into
Open
Conversation
bruno-garcia
force-pushed
the
deps/modules/sentry-native/0.16.4
branch
from
August 26, 2026 03:19
9f113e1 to
4cd8f7a
Compare
| @@ -1 +1 @@ | |||
| Subproject commit 724479b549a299ea8363994306b36a00c754fcba | |||
| Subproject commit 22e1073a32e940b190e048890ede26c1dd1db3ef | |||
There was a problem hiding this comment.
Bug: The update to sentry-native now dynamically loads libcurl by default. The build doesn't bundle libcurl or force static linking, which may cause silent failures on minimal Linux systems.
Severity: MEDIUM
Suggested Fix
To ensure compatibility with environments that may not have libcurl installed, explicitly configure the sentry-native build to statically link libcurl. In the build/native-sdks.targets file, add -D SENTRY_LINK_CURL=ON to the cmake command for Linux builds. This will restore the previous behavior of bundling the library and avoid runtime dependency issues.
Prompt for AI Agent
Review the code at the location below. A potential bug has been identified by an AI
agent. Verify if this is a real issue. If it is, propose a fix; if not, explain why it's
not valid.
Location: modules/sentry-native#L1
Potential issue: The `sentry-native` submodule was updated, changing the default
`SENTRY_LINK_CURL` setting to `AUTO`. This causes `libcurl` to be dynamically loaded at
runtime on Linux instead of being statically linked. However, the Unity SDK's build
process does not bundle `libcurl.so` with the application artifacts, nor does it
override the new default to force static linking. Consequently, if a game is deployed to
a minimal Linux environment (e.g., a Docker container) that lacks a system-installed
`libcurl`, the dynamic load will fail. This will likely result in silent failure of
Sentry event and crash reporting, as the transport mechanism cannot be initialized.
Did we get this right? 👍 / 👎 to inform future reviews.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Bumps modules/sentry-native from 0.16.2 to 0.16.4.
Auto-generated by a dependency updater.
Changelog
0.16.4
Features:
sentry_http_transport_new) so applications can plug in their own HTTP client (e.g. platform-native ones) while sentry-native continues to own request queueing, retry/backoff, offline caching, rate-limiting, and client reports. The built-in curl and WinHTTP transports are now implemented against this same interface. (#1987)contexts.wer.report_idin crash events when the WER integration is enabled. (#1970)sentry_set_tagsandsentry_scope_set_tagsfor updating multiple tags with a single scope flush, improving bulk-update performance. (#1993)on_crashed_last_runcallback for inspecting crash envelopes from previous runs. (#1985)sentry_get_last_event_idandsentry_scope_get_last_event_idfor retrieving the last event ID captured with the global or given scope, respectively. (#1992)libcurldynamically at runtime by default whenSENTRY_LINK_CURL=AUTO, avoidinglibcurllinker work during process startup and significantly speeding up startup time. Explicitly setSENTRY_LINK_CURL=ONto link it directly. (#1955)set_release,set_environment,set_transaction,remove_tag,remove_extra,remove_context, andremove_attachment. (#2011)Deprecations:
sentry_options_get/set_enable_logsandsentry_options_get/set_enable_metrics. (#2000)Fixes:
.rundirectories so they are cleaned up with the run instead of accumulating in the database root. Minidumps can still be retained withcache_keep, which stores.dmpsidecars alongside cached envelopes. (#1976)0.16.3
Features:
enable_logsoption throughNdkOptionsso the Android SDK can enable native structured logging. (#1971)sdk.integrations) alongside the configured crash backend. ([fix: Flaky iOS smoke tests #1969](https://github-redirect.dependabot.com/feat: report SDK integrations sentry-native#1969))
sentry_scope_set_spanandsentry_scope_set_transaction_objectto bind a span or transaction to a given scope. (#1981)Fixes:
serializing envelope into bufferdebug log that could flood logs on every scope flush when an external crash reporter is configured. (#1988)base::FilePathinside the crash handler; if the crash corrupted the heap (e.g. via an overriddenoperator new) that allocation faults again and the report is lost. The paths are now cached at startup. (#1984)NULbytes in strings created withsentry_value_new_string_nduring JSON and MessagePack serialization.sentry_value_get_lengthnow returns the full byte length, which can exceedstrlen(sentry_value_as_string(value)). (#1968)