Skip to content

Commit 17a455f

Browse files
committed
fix: request logging
1 parent 19dfb92 commit 17a455f

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

src/teapot.cpp

+5-5
Original file line numberDiff line numberDiff line change
@@ -71,6 +71,8 @@ void Teapot::requestHandler(SOCKET client_socket)
7171
auto request = parseRequest(client_socket);
7272
std::string body;
7373
std::string content_type;
74+
std::string method;
75+
std::string uri;
7476
unsigned int status_code = 500; // Default to internal server error in case of early failure
7577

7678
try
@@ -79,14 +81,12 @@ void Teapot::requestHandler(SOCKET client_socket)
7981
{
8082
context->request = &(*request);
8183
this->sanitizer_middleware.handle(context.get());
82-
std::string uri = request->getUri();
84+
uri = request->getUri();
8385
if (uri == "/")
8486
uri = "/index.html"; // Normalize root access to a specific file, e.g., index.html
85-
std::string method = request->getMethod();
87+
method = request->getMethod();
8688
content_type = determineContentType(uri); // Determine content type early based on URI
8789

88-
std::cout << "[" << request->getDate() << "] " << this->socket.getClientIp() + " " << method << " " << uri << " HTTP/1.1 ";
89-
9090
if (method == "GET")
9191
{
9292
// Check for predefined routes or responses before attempting to read a file
@@ -151,7 +151,7 @@ void Teapot::requestHandler(SOCKET client_socket)
151151
this->cors_middleware.handle(context.get());
152152
this->security_middleware.handle(context.get());
153153

154-
std::cout << response.getStatusCode() + " " + response.getStatusCodeDescription() << "\n";
154+
LOG_INFO(logger, this->socket.getClientIp() + " " + method + " " + uri + " HTTP/1.1 " + response.getStatusCode() + " " + response.getStatusCodeDescription());
155155

156156
std::string raw_response = response.getRawResponse();
157157
this->socket.sendData(client_socket, raw_response.c_str(), raw_response.length(), 0);

0 commit comments

Comments
 (0)