Set log levels based on build type

CMakeLists.txt

@@ -3,6 +3,16 @@ project(cpp_redis)
 
 set(CMAKE_CXX_STANDARD 20)
 
+include(FetchContent)
+
+# TODO: create separate CMAKE for dependencies
+FetchContent_Declare(
+        spdlog
+        GIT_REPOSITORY https://github.com/gabime/spdlog.git
+        GIT_TAG v1.14.1
+)
+FetchContent_MakeAvailable(spdlog)
+
 add_subdirectory(src)
 add_subdirectory(tests)
 

src/CMakeLists.txt

@@ -10,4 +10,10 @@ add_executable(cpp_redis main.cpp
         datastore.cpp)
 
 
+if (CMAKE_BUILD_TYPE STREQUAL "Debug")
+    target_compile_definitions(cpp_redis PRIVATE SPDLOG_ACTIVE_LEVEL=SPDLOG_LEVEL_DEBUG)
+elseif (CMAKE_BUILD_TYPE STREQUAL "Release")
+    target_compile_definitions(cpp_redis PRIVATE SPDLOG_ACTIVE_LEVEL=SPDLOG_LEVEL_INFO)
+endif ()
+
 target_link_libraries(cpp_redis PRIVATE spdlog::spdlog_header_only)

src/main.cpp

@@ -1,8 +1,13 @@
-#include <iostream>
-
+#include "spdlog/spdlog.h"
 #include "tcp_server.h"
 
 int main() {
+#if SPDLOG_ACTIVE_LEVEL == SPDLOG_LEVEL_DEBUG
+    spdlog::set_level(spdlog::level::debug);
+#elif SPDLOG_ACTIVE_LEVEL == SPDLOG_LEVEL_INFO
+    spdlog::set_level(spdlog::level::info);
+#endif
+
     TCPServer server;
     server.start("0.0.0.0", 6379);
 }

src/tcp_server.cpp

@@ -25,7 +25,6 @@ TCPServer::TCPServer() : controller{} {
     }
 }
 
-// TODO: why can it start if the port is already taken?
 [[noreturn]] void TCPServer::start(const std::string &address = "0.0.0.0", int port = 6379) {
     struct sockaddr_in server_addr {};
     server_addr.sin_family = AF_INET;
@@ -36,7 +35,7 @@ TCPServer::TCPServer() : controller{} {
         throw std::runtime_error("Port " + std::to_string(port) + " already in use!");
     }
 
-    int connection_backlog = 5;
+    int connection_backlog = 128;
     if (listen(m_serverFD, connection_backlog) != 0) { throw std::runtime_error("Listen failed!"); }
 
     spdlog::info("Listening on port {}", port);
@@ -118,7 +117,7 @@ void TCPServer::handleRequest(int connFD) {
         // Handle command
         RedisType::RedisValue res = controller.handleCommand(command);
         auto encoded = encode(res);
-        spdlog::info("Request: {}, Response: {}", std::get<RedisType::Array>(message), res);
+        spdlog::debug("Request: {}, Response: {}", std::get<RedisType::Array>(message), res);
 
         // Send response
         send(connFD, encoded.data(), encoded.size(), 0);

src/tcp_server.h

@@ -1,6 +1,7 @@
 #pragma once
 
 #include "controller.h"
+#include <atomic>
 #include <string>
 
 
@@ -13,7 +14,7 @@ class TCPServer {
 private:
     int m_serverFD;
     Controller controller;
-
+    
     static constexpr size_t RECV_SIZE = 2048;
 };
 

tests/CMakeLists.txt

@@ -8,13 +8,6 @@ FetchContent_Declare(
 
 FetchContent_MakeAvailable(googletest)
 
-FetchContent_Declare(
-        spdlog
-        GIT_REPOSITORY https://github.com/gabime/spdlog.git
-        GIT_TAG v1.14.1
-)
-FetchContent_MakeAvailable(spdlog)
-
 add_library(GTest::GTest INTERFACE IMPORTED)
 target_link_libraries(GTest::GTest INTERFACE gtest_main)