search

getsentry / sentry-native

Sentry SDK for C, C++ and native applications.
MIT License
403 stars 170 forks source link

feat: expose function to send minidumps #1067

Closed vaind closed 3 weeks ago

vaind commented 3 weeks ago

Adds sentry_capture_minidump(const char*) to send an existing minidump.

This PR builds on top of #1052 but makes significant API changes so I've made a new PR instead of rewriting the old one.

Note: CI failures seem to be unrelated, see https://github.com/getsentry/sentry-native/issues/1069

github-actions[bot] commented 3 weeks ago
Messages
:book: Do not forget to update Sentry-docs with your feature once the pull request gets approved.

Generated by :no_entry_sign: dangerJS against 40ec96e89617f3bd89965d55274f273cb6590ab3

bruno-garcia commented 3 weeks ago

Thank you @vaind !

codecov[bot] commented 3 weeks ago

Codecov Report

Attention: Patch coverage is 89.18919% with 4 lines in your changes missing coverage. Please review.

Project coverage is 82.62%. Comparing base (ffdf6ed) to head (40ec96e). Report is 1 commits behind head on master.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #1067 +/- ## ========================================== - Coverage 82.65% 82.62% -0.04% ========================================== Files 53 53 Lines 7680 7717 +37 Branches 1209 1213 +4 ========================================== + Hits 6348 6376 +28 - Misses 1219 1228 +9 Partials 113 113 ```