Skip to content

fix(native): Initialize null transport in crash daemon - #2049

Merged
jpnurmi merged 5 commits into
masterfrom
jpnurmi/fix/daemon-null-transport
Sep 3, 2026
Merged

fix(native): Initialize null transport in crash daemon #2049
jpnurmi merged 5 commits into
masterfrom
jpnurmi/fix/daemon-null-transport

Conversation

@jpnurmi

@jpnurmi jpnurmi commented Sep 2, 2026

Copy link
Copy Markdown
Collaborator

The daemon constructs options without calling sentry_init(), so builds with SENTRY_TRANSPORT=none leave the transport pointer unset. Install the internal null transport so crash processing completes without leaving stale envelopes on disk.

The daemon constructs options without calling sentry_init(), so builds with
SENTRY_TRANSPORT=none leave the transport pointer unset. Install the internal
null transport so crash processing completes without leaving stale envelopes
on disk.
@codecov

codecov Bot commented Sep 2, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.19%. Comparing base (7770d2e) to head (ad57075).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2049      +/-   ##
==========================================
- Coverage   74.31%   74.19%   -0.12%     
==========================================
  Files         103      103              
  Lines       26661    26664       +3     
  Branches     4849     4850       +1     
==========================================
- Hits        19813    19784      -29     
- Misses       5519     5554      +35     
+ Partials     1329     1326       -3     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@jpnurmi
jpnurmi merged commit fd479c0 into master Sep 3, 2026
71 checks passed
@jpnurmi
jpnurmi deleted the jpnurmi/fix/daemon-null-transport branch September 3, 2026 20: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.

3 participants