willem/webserv
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

formatting

Browse Source
This commit is contained in:
Quinten 2025-10-29 15:51:32 +01:00
parent e1e3003d94
commit e34205bd39
1 changed files with 34 additions and 34 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

66
webserv/client/Client.cpp
View File

@ -87,40 +87,39 @@ void Client::request()
Log::info("Received request: " + httpRequest_->getHttpVersion() + " " + httpRequest_->getMethod() + " "
+ httpRequest_->getTarget() + " ");
Log::debug("Request details",
{
{"request_method", httpRequest_->getMethod()},
{"request_target", httpRequest_->getTarget()},
{"http_version", httpRequest_->getHttpVersion()},
{"headers", httpRequest_->getHeaders().toString()},
{"body", httpRequest_->getBody()},
{"state", std::to_string(static_cast<uint8_t>(httpRequest_->getState()))},
});
Log::debug("Request details", {
{"request_method", httpRequest_->getMethod()},
{"request_target", httpRequest_->getTarget()},
{"http_version", httpRequest_->getHttpVersion()},
{"headers", httpRequest_->getHeaders().toString()},
// {"body", httpRequest_->getBody()},
{"state", std::to_string(static_cast<uint8_t>(httpRequest_->getState()))},
});
try
{
// Thoughts: if a handler isn't returned, this could because of the error handler already setting
// up the response so, maybe we don't need to throw a 500 when no handler. Because that would
// override the actual error response. How about the router, or a handler, throws an exception if
// something goes wrong, and we catch it here to make a 500 response?
handler_ = router_->handleRequest();
if (handler_ != nullptr)
{
handler_->handle();
}
}
catch (const RequestValidator::ValidationException &e)
{
Log::error("Validation Exception during request handling: " + std::string(e.what()));
ErrorHandler::createErrorResponse(e.code(), *httpResponse_);
return;
}
catch (const std::exception &e)
{
Log::error("Exception during request handling: " + std::string(e.what()));
ErrorHandler::createErrorResponse(500, *httpResponse_);
return;
}
try
{
// Thoughts: if a handler isn't returned, this could because of the error handler already setting
// up the response so, maybe we don't need to throw a 500 when no handler. Because that would
// override the actual error response. How about the router, or a handler, throws an exception if
// something goes wrong, and we catch it here to make a 500 response?
handler_ = router_->handleRequest();
if (handler_ != nullptr)
{
handler_->handle();
}
}
catch (const RequestValidator::ValidationException &e)
{
Log::error("Validation Exception during request handling: " + std::string(e.what()));
ErrorHandler::createErrorResponse(e.code(), *httpResponse_);
return;
}
catch (const std::exception &e)
{
Log::error("Exception during request handling: " + std::string(e.what()));
ErrorHandler::createErrorResponse(500, *httpResponse_);
return;
}
}
else
{
@ -200,4 +199,5 @@ std::string Client::getClientAddress() const noexcept
return "";
}
// NOLINTEND