rm unused/broken log_sink for future proofing

Author: stephen-derosa

docs/doxygen/index.md

@@ -9,7 +9,7 @@ Build real-time audio/video applications in C++ with LiveKit.
 
 bool initializeLivekit(const std::string& url, const std::string& token) {
   // Init LiveKit
-  livekit::initialize(livekit::LogLevel::Info, livekit::LogSink::kConsole);
+  livekit::initialize(livekit::LogLevel::Info);
 
   room_ = std::make_unique<livekit::Room>();
   livekit::RoomOptions options;

include/livekit/livekit.h

@@ -44,25 +44,16 @@
 /// @brief Public API for the LiveKit C++ Client SDK.
 namespace livekit {
 
-/// The log sink to use for SDK messages.
-enum class LogSink {
-  /// Log messages to the console.
-  kConsole = 0,
-  /// Log messages to a callback function.
-  kCallback = 1,
-};
-
 /// Initialize the LiveKit SDK.
 ///
 /// This **must be the first LiveKit API called** in the process.
 /// It configures global SDK state, including log routing.
 ///
 /// @param level     Minimum log level for SDK messages (default: Info).
 ///                  Use setLogLevel() to change at runtime.
-/// @param log_sink  The log sink to use for SDK messages (default: Console).
 /// @returns true if initialization happened on this call, false if it was
 ///          already initialized.
-LIVEKIT_API bool initialize(const LogLevel& level = LogLevel::Info, const LogSink& log_sink = LogSink::kConsole);
+LIVEKIT_API bool initialize(const LogLevel& level = LogLevel::Info);
 
 /// Shut down the LiveKit SDK.
 ///

src/livekit.cpp

@@ -21,11 +21,12 @@
 
 namespace livekit {
 
-bool initialize(const LogLevel& level, const LogSink& log_sink) {
+bool initialize(const LogLevel& level) {
   // Initializes logger if singleton instance is not already initialized
   setLogLevel(level);
   auto& ffi_client = FfiClient::instance();
-  return ffi_client.initialize(log_sink == LogSink::kCallback);
+  // Note: capture_logs currently disabled, requires event support in FfiClient
+  return ffi_client.initialize(false);
 }
 
 bool isInitialized() { return FfiClient::instance().isInitialized(); }

src/tests/common/test_common.h

@@ -497,7 +497,7 @@ class StressTestStats {
 class LiveKitTestBase : public ::testing::Test {
 protected:
   void SetUp() override {
-    livekit::initialize(livekit::LogLevel::Info, livekit::LogSink::kConsole);
+    livekit::initialize(livekit::LogLevel::Info);
     config_ = TestConfig::fromEnv();
 
     // Tracing is controlled by compile-time macro LIVEKIT_TEST_ENABLE_TRACING

src/tests/integration/test_room.cpp

@@ -29,7 +29,7 @@ namespace livekit::test {
 class RoomTest : public ::testing::Test {
 protected:
   void SetUp() override {
-    livekit::initialize(livekit::LogLevel::Info, livekit::LogSink::kConsole);
+    livekit::initialize(livekit::LogLevel::Info);
 
     const char* url_env = std::getenv("LIVEKIT_URL");
     const char* token_env = std::getenv("LIVEKIT_TOKEN_A");

src/tests/integration/test_rpc.cpp

@@ -93,7 +93,7 @@ std::string generateRandomPayload(size_t size) {
 class RpcIntegrationTest : public ::testing::Test {
 protected:
   void SetUp() override {
-    livekit::initialize(livekit::LogLevel::Info, livekit::LogSink::kConsole);
+    livekit::initialize(livekit::LogLevel::Info);
     config_ = RpcTestConfig::fromEnv();
   }
 

src/tests/stress/test_audio_frame_stress.cpp

@@ -27,7 +27,7 @@ namespace livekit::test {
 
 class AudioFrameStressTest : public ::testing::Test {
 protected:
-  void SetUp() override { livekit::initialize(livekit::LogLevel::Info, livekit::LogSink::kConsole); }
+  void SetUp() override { livekit::initialize(livekit::LogLevel::Info); }
 
   void TearDown() override { livekit::shutdown(); }
 };

src/tests/stress/test_room_stress.cpp

@@ -27,7 +27,7 @@ namespace livekit::test {
 
 class RoomStressTest : public ::testing::Test {
 protected:
-  void SetUp() override { livekit::initialize(livekit::LogLevel::Info, livekit::LogSink::kConsole); }
+  void SetUp() override { livekit::initialize(livekit::LogLevel::Info); }
 
   void TearDown() override { livekit::shutdown(); }
 };
@@ -176,7 +176,7 @@ TEST_F(RoomStressTest, StreamHandlerRegistrationStress) {
 class RoomServerStressTest : public ::testing::Test {
 protected:
   void SetUp() override {
-    livekit::initialize(livekit::LogLevel::Info, livekit::LogSink::kConsole);
+    livekit::initialize(livekit::LogLevel::Info);
 
     const char* url_env = std::getenv("LIVEKIT_URL");
     const char* token_env = std::getenv("LIVEKIT_TOKEN_A");

src/tests/unit/test_audio_frame.cpp

@@ -22,7 +22,7 @@ namespace livekit::test {
 
 class AudioFrameTest : public ::testing::Test {
 protected:
-  void SetUp() override { livekit::initialize(livekit::LogLevel::Info, livekit::LogSink::kConsole); }
+  void SetUp() override { livekit::initialize(livekit::LogLevel::Info); }
 
   void TearDown() override { livekit::shutdown(); }
 };

src/tests/unit/test_audio_processing_module.cpp

@@ -32,7 +32,7 @@ namespace livekit::test {
 
 class AudioProcessingModuleTest : public ::testing::Test {
 protected:
-  void SetUp() override { livekit::initialize(livekit::LogLevel::Info, livekit::LogSink::kConsole); }
+  void SetUp() override { livekit::initialize(livekit::LogLevel::Info); }
 
   void TearDown() override { livekit::shutdown(); }