Skip to content

Commit 0c2cbfe

Browse files
committed
Add debug logging
1 parent d3b18cd commit 0c2cbfe

3 files changed

Lines changed: 12 additions & 5 deletions

File tree

libs/networking/src/curl_multi_manager.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -196,7 +196,7 @@ void CurlMultiManager::check_multi_info() {
196196
// Invoke completion callback
197197
if (callback) {
198198
boost::asio::post(executor_, [callback = std::move(callback),
199-
easy, result, handle]() {
199+
result, handle]() {
200200
callback(handle, result);
201201
});
202202
}

libs/server-sent-events/src/curl_client.cpp

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -326,6 +326,7 @@ int CurlClient::ProgressCallback(void* clientp,
326326
now - context->last_progress_time);
327327

328328
if (elapsed > *context->read_timeout) {
329+
context->log_message("Read timeout, aborting curl transfer.");
329330
return kCurlTransferAbort;
330331
}
331332
}

libs/server-sent-events/tests/curl_client_test.cpp

Lines changed: 10 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -640,22 +640,28 @@ TEST(CurlClientTest, RespectsReadTimeout) {
640640

641641
auto client = Builder(runner.context().get_executor(), "http://localhost:" + std::to_string(port))
642642
.receiver([&](Event e) { collector.add_event(std::move(e)); })
643-
.errors([&](Error e) { collector.add_error(std::move(e)); })
643+
.errors([&](Error e) {
644+
std::cerr << "Error" << e.index() << std::endl;
645+
collector.add_error(std::move(e));
646+
})
647+
.logger([&](const std::string& message) {
648+
std::cerr << "log_message" << message << std::endl;
649+
})
644650
.read_timeout(500ms) // Short timeout for test
645651
.initial_reconnect_delay(50ms)
646652
.build();
647653

648654
client->async_connect();
649655

650656
// Should receive the first event
651-
ASSERT_TRUE(collector.wait_for_events(1, 2000ms));
657+
ASSERT_TRUE(collector.wait_for_events(1, 100ms));
652658

653659
// Then should get a timeout error
654-
ASSERT_TRUE(collector.wait_for_errors(1, 3000ms));
660+
ASSERT_TRUE(collector.wait_for_errors(1, 1000ms));
655661

656662
SimpleLatch shutdown_latch(1);
657663
client->async_shutdown([&] { shutdown_latch.count_down(); });
658-
EXPECT_TRUE(shutdown_latch.wait_for(5000ms));
664+
EXPECT_TRUE(shutdown_latch.wait_for(100ms));
659665
}
660666

661667
// Resource management tests

0 commit comments

Comments
 (0)