diff --git a/src/sentry_logger.c b/src/sentry_logger.c index 7634a4831..ab6bd05c4 100644 --- a/src/sentry_logger.c +++ b/src/sentry_logger.c @@ -53,7 +53,7 @@ sentry__logger_defaultlogger( size_t len = strlen(prefix) + strlen(priority) + strlen(message) + 2; char *format = sentry_malloc(len); - snprintf("%s%s%s\n", len, prefix, priority, message); + snprintf(format, len, "%s%s%s\n", prefix, priority, message); vfprintf(stderr, format, args); diff --git a/tests/test_integration_http.py b/tests/test_integration_http.py index a28cdfc6e..8e96b566a 100644 --- a/tests/test_integration_http.py +++ b/tests/test_integration_http.py @@ -35,7 +35,7 @@ def test_capture_http(cmake, httpserver): run( tmp_path, "sentry_example", - ["release-env", "capture-event", "add-stacktrace"], + ["log", "release-env", "capture-event", "add-stacktrace"], check=True, env=env, ) @@ -63,14 +63,14 @@ def test_session_http(cmake, httpserver): run( tmp_path, "sentry_example", - ["release-env", "start-session"], + ["log", "release-env", "start-session"], check=True, env=dict(os.environ, SENTRY_DSN=make_dsn(httpserver)), ) run( tmp_path, "sentry_example", - ["start-session"], + ["log", "start-session"], check=True, env=dict(os.environ, SENTRY_DSN=make_dsn(httpserver)), ) @@ -92,7 +92,7 @@ def test_capture_and_session_http(cmake, httpserver): run( tmp_path, "sentry_example", - ["start-session", "capture-event"], + ["log", "start-session", "capture-event"], check=True, env=dict(os.environ, SENTRY_DSN=make_dsn(httpserver)), ) @@ -113,7 +113,9 @@ def test_capture_and_session_http(cmake, httpserver): def test_inproc_crash_http(cmake, httpserver): tmp_path = cmake(["sentry_example"], {"SENTRY_BACKEND": "inproc"}) - child = run(tmp_path, "sentry_example", ["start-session", "attachment", "crash"]) + child = run( + tmp_path, "sentry_example", ["log", "start-session", "attachment", "crash"] + ) assert child.returncode # well, its a crash after all httpserver.expect_request( @@ -123,7 +125,7 @@ def test_inproc_crash_http(cmake, httpserver): run( tmp_path, "sentry_example", - ["no-setup"], + ["log", "no-setup"], check=True, env=dict(os.environ, SENTRY_DSN=make_dsn(httpserver)), ) @@ -154,10 +156,12 @@ def test_inproc_dump_inflight(cmake, httpserver): ).respond_with_data("OK") env = dict(os.environ, SENTRY_DSN=make_dsn(httpserver)) - child = run(tmp_path, "sentry_example", ["capture-multiple", "crash"], env=env) + child = run( + tmp_path, "sentry_example", ["log", "capture-multiple", "crash"], env=env + ) assert child.returncode # well, its a crash after all - run(tmp_path, "sentry_example", ["no-setup"], check=True, env=env) + run(tmp_path, "sentry_example", ["log", "no-setup"], check=True, env=env) # we trigger 10 normal events, and 1 crash assert len(httpserver.log) >= 11 @@ -167,7 +171,9 @@ def test_inproc_dump_inflight(cmake, httpserver): def test_breakpad_crash_http(cmake, httpserver): tmp_path = cmake(["sentry_example"], {"SENTRY_BACKEND": "breakpad"}) - child = run(tmp_path, "sentry_example", ["start-session", "attachment", "crash"]) + child = run( + tmp_path, "sentry_example", ["log", "start-session", "attachment", "crash"] + ) assert child.returncode # well, its a crash after all httpserver.expect_request( @@ -177,7 +183,7 @@ def test_breakpad_crash_http(cmake, httpserver): run( tmp_path, "sentry_example", - ["no-setup"], + ["log", "no-setup"], check=True, env=dict(os.environ, SENTRY_DSN=make_dsn(httpserver)), ) @@ -209,10 +215,12 @@ def test_breakpad_dump_inflight(cmake, httpserver): ).respond_with_data("OK") env = dict(os.environ, SENTRY_DSN=make_dsn(httpserver)) - child = run(tmp_path, "sentry_example", ["capture-multiple", "crash"], env=env) + child = run( + tmp_path, "sentry_example", ["log", "capture-multiple", "crash"], env=env + ) assert child.returncode # well, its a crash after all - run(tmp_path, "sentry_example", ["no-setup"], check=True, env=env) + run(tmp_path, "sentry_example", ["log", "no-setup"], check=True, env=env) # we trigger 10 normal events, and 1 crash assert len(httpserver.log) >= 11 diff --git a/tests/test_integration_ratelimits.py b/tests/test_integration_ratelimits.py index a64cb2c77..e265d3c58 100644 --- a/tests/test_integration_ratelimits.py +++ b/tests/test_integration_ratelimits.py @@ -16,13 +16,13 @@ def test_retry_after(cmake, httpserver): httpserver.expect_oneshot_request("/api/123456/envelope/").respond_with_data( "OK", 200, {"retry-after": "60"} ) - run(tmp_path, "sentry_example", ["capture-multiple"], check=True, env=env) + run(tmp_path, "sentry_example", ["log", "capture-multiple"], check=True, env=env) assert len(httpserver.log) == 1 httpserver.expect_oneshot_request("/api/123456/envelope/").respond_with_data( "OK", 429, {"retry-after": "60"} ) - run(tmp_path, "sentry_example", ["capture-multiple"], check=True, env=env) + run(tmp_path, "sentry_example", ["log", "capture-multiple"], check=True, env=env) assert len(httpserver.log) == 2 @@ -35,11 +35,11 @@ def test_rate_limits(cmake, httpserver): httpserver.expect_oneshot_request("/api/123456/envelope/").respond_with_data( "OK", 200, headers ) - run(tmp_path, "sentry_example", ["capture-multiple"], check=True, env=env) + run(tmp_path, "sentry_example", ["log", "capture-multiple"], check=True, env=env) assert len(httpserver.log) == 1 httpserver.expect_oneshot_request("/api/123456/envelope/").respond_with_data( "OK", 429, headers ) - run(tmp_path, "sentry_example", ["capture-multiple"], check=True, env=env) + run(tmp_path, "sentry_example", ["log", "capture-multiple"], check=True, env=env) assert len(httpserver.log) == 2