Skip to content

Commit 985abed

Browse files
committed
DEBUG
1 parent 7f1687c commit 985abed

4 files changed

Lines changed: 383 additions & 14 deletions

File tree

.github/workflows/ci.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -467,7 +467,7 @@ jobs:
467467
shell: bash
468468
run: |
469469
pip install --upgrade --requirement tests/requirements.txt
470-
pytest --capture=no --verbose tests
470+
pytest --capture=no --verbose tests/test_integration_native.py
471471
472472
- name: Check NDK
473473
if: ${{ env['ANDROID_API'] }}

src/backends/native/sentry_crash_daemon.c

Lines changed: 28 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -60,6 +60,21 @@
6060
# include <sys/stat.h>
6161
# include <windows.h>
6262

63+
static void
64+
daemon_debugf(const char *fmt, ...)
65+
{
66+
char msg[1024];
67+
va_list args;
68+
va_start(args, fmt);
69+
int n = vsnprintf(msg, sizeof(msg), fmt, args);
70+
va_end(args);
71+
if (n < 0) {
72+
return;
73+
}
74+
msg[sizeof(msg) - 1] = '\0';
75+
OutputDebugStringA(msg);
76+
}
77+
6378
// Forward declaration for StackWalk64-based stack unwinding (defined later)
6479
static size_t walk_stack_with_dbghelp(HANDLE hProcess, DWORD crashed_tid,
6580
const CONTEXT *ctx_record, sentry_frame_info_t *frames, size_t max_frames);
@@ -2240,6 +2255,7 @@ static bool
22402255
add_wer_context(sentry_value_t event, const sentry_crash_context_t *ctx)
22412256
{
22422257
if (!ctx->platform.wer_enabled) {
2258+
daemon_debugf("### DAEMON: add_wer_context skipped wer_enabled=0\n");
22432259
return false;
22442260
}
22452261

@@ -2248,12 +2264,23 @@ add_wer_context(sentry_value_t event, const sentry_crash_context_t *ctx)
22482264
wer_report_id, ctx->platform.wer_report_id, sizeof(wer_report_id) - 1);
22492265
wer_report_id[sizeof(wer_report_id) - 1] = '\0';
22502266

2267+
daemon_debugf(
2268+
"### DAEMON: add_wer_context start event_id=%s report_id=%s\n",
2269+
ctx->crash_event_id,
2270+
sentry__string_empty(wer_report_id) ? "(empty)" : wer_report_id);
2271+
22512272
sentry_value_t wer_context = sentry__wer_report_lookup(ctx->crash_event_id);
22522273
if (sentry_value_is_null(wer_context)) {
2274+
daemon_debugf("### DAEMON: WER metadata xml not found yet\n");
22532275
if (sentry__string_empty(wer_report_id)) {
2276+
daemon_debugf(
2277+
"### DAEMON: no fallback report_id, skipping wer context\n");
22542278
return false;
22552279
}
2280+
daemon_debugf("### DAEMON: using fallback report_id from callback\n");
22562281
wer_context = sentry_value_new_object();
2282+
} else {
2283+
daemon_debugf("### DAEMON: WER metadata xml found\n");
22572284
}
22582285

22592286
sentry_value_set_by_key(
@@ -2271,6 +2298,7 @@ add_wer_context(sentry_value_t event, const sentry_crash_context_t *ctx)
22712298
sentry_value_set_by_key(event, "contexts", contexts);
22722299
}
22732300
sentry_value_set_by_key(contexts, "wer", wer_context);
2301+
daemon_debugf("### DAEMON: add_wer_context attached wer context\n");
22742302
return true;
22752303
}
22762304
#endif

0 commit comments

Comments
 (0)