prioritize router matching, unfold ifs
This commit is contained in:
parent
72d17e1623
commit
a4a5d580f6
15
server.hpp
15
server.hpp
@ -152,11 +152,6 @@ private:
|
|||||||
|
|
||||||
Response response;
|
Response response;
|
||||||
|
|
||||||
if (static_handler && static_handler->handle(req, response)) {
|
|
||||||
send_response(client_fd, response, req.version);
|
|
||||||
return;
|
|
||||||
}
|
|
||||||
|
|
||||||
if (router.handle(req, response)) {
|
if (router.handle(req, response)) {
|
||||||
std::string_view existing_id = sessions.extract_session_id(req);
|
std::string_view existing_id = sessions.extract_session_id(req);
|
||||||
std::string session_id = existing_id.empty() ?
|
std::string session_id = existing_id.empty() ?
|
||||||
@ -164,12 +159,18 @@ private:
|
|||||||
|
|
||||||
set_session_cookie(response, session_id);
|
set_session_cookie(response, session_id);
|
||||||
send_response(client_fd, response, req.version);
|
send_response(client_fd, response, req.version);
|
||||||
} else {
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
if (static_handler && static_handler->handle(req, response)) {
|
||||||
|
send_response(client_fd, response, req.version);
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
response.status = 404;
|
response.status = 404;
|
||||||
response.set_text("Not Found");
|
response.set_text("Not Found");
|
||||||
send_response(client_fd, response, req.version);
|
send_response(client_fd, response, req.version);
|
||||||
}
|
}
|
||||||
}
|
|
||||||
|
|
||||||
void set_session_cookie(Response& response, const std::string& session_id) {
|
void set_session_cookie(Response& response, const std::string& session_id) {
|
||||||
response.cookies.push_back("session_id=" + session_id + "; HttpOnly; Path=/; SameSite=Strict");
|
response.cookies.push_back("session_id=" + session_id + "; HttpOnly; Path=/; SameSite=Strict");
|
||||||
|
Loading…
x
Reference in New Issue
Block a user