refactor(exceptions): #25: add ExceptionResponseHandler to handle exceptions

src/api/exception-response-handler.cpp

@@ -0,0 +1,22 @@
+#include "exception-response-handler.h"
+
+#include <drogon/drogon.h>
+
+drogon::HttpResponsePtr ExceptionResponseHandler::handle(const std::string& status, const std::string& message, drogon::HttpStatusCode status_code)
+{
+    Json::Value responseBody;
+    responseBody["status"] = status;
+    responseBody["message"] = message;
+
+    auto response = drogon::HttpResponse::newHttpJsonResponse(responseBody);
+    response->setStatusCode(status_code);
+
+    return response;
+}
+
+drogon::HttpResponsePtr ExceptionResponseHandler::handle(const std::exception& exception)
+{
+    LOG_ERROR << "Exception occured: " << exception.what();
+
+    return ExceptionResponseHandler::handle("error", "Internal server error", drogon::k500InternalServerError);
+}

src/api/exception-response-handler.h

@@ -0,0 +1,21 @@
+#pragma once
+
+#include <drogon/HttpResponse.h>
+
+#include <exception>
+
+class ExceptionResponseHandler
+{
+public:
+    /// @brief Creates a JSON HTTP error response with the specified status and message.
+    /// @param status Status value included in the JSON response body.
+    /// @param message Message included in the JSON response body.
+    /// @param status_code HTTP status code returned to the client.
+    /// @return JSON HTTP response.
+    [[nodiscard]] static drogon::HttpResponsePtr handle(const std::string& status, const std::string& message, drogon::HttpStatusCode status_code);
+
+    /// @brief Handles an unexpected exception and creates an internal server error response.
+    /// @param exception Exception caught while handling an HTTP request.
+    /// @return HTTP 500 JSON response.
+    [[nodiscard]] static drogon::HttpResponsePtr handle(const std::exception& exception);
+};

src/api/features/to-dos/to-dos-controller.cpp

@@ -1,18 +1,7 @@
 #include "to-dos-controller.h"
 #include "db_connection.h"
+#include <exception-response-handler.h>
 
-// TODO(https://github.com/TourmalineCore/to-dos-api-cpp/issues/25): Create http exception handler or generic class for handling that type of errors
-HttpResponsePtr ToDosController::createInternalServerErrorResponse(const std::string& error) const
-{
-    Json::Value jsonResponse;
-    jsonResponse["status"] = "Error";
-    jsonResponse["message"] = "Internal server error";
-    jsonResponse["error"] = error;
-
-    auto resp = HttpResponse::newHttpJsonResponse(jsonResponse);
-    resp->setStatusCode(k500InternalServerError);
-    return resp;
-}
 
 ToDosController::ToDosController()
 {
@@ -48,9 +37,9 @@ void ToDosController::getToDos(const HttpRequestPtr& req, std::function<void(con
         resp->setStatusCode(k200OK);
         callback(resp);
     }
-    catch (const std::exception& e)
+    catch (const std::exception& exception)
     {
-        callback(createInternalServerErrorResponse(e.what()));
+        callback(ExceptionResponseHandler::handle(exception));
     }
 }
 
@@ -79,9 +68,9 @@ void ToDosController::addToDo(const HttpRequestPtr& req, std::function<void(cons
         resp->setStatusCode(k201Created);
         callback(resp);
     }
-    catch (const std::exception& e)
+    catch (const std::exception& exception)
     {
-        callback(createInternalServerErrorResponse(e.what()));
+        callback(ExceptionResponseHandler::handle(exception));
     }
 }
 
@@ -114,9 +103,9 @@ void ToDosController::completeToDos(const HttpRequestPtr& req, std::function<voi
         resp->setStatusCode(k200OK);
         callback(resp);
     }
-    catch (const std::exception& e)
+    catch (const std::exception& exception)
     {
-        callback(createInternalServerErrorResponse(e.what()));
+        callback(ExceptionResponseHandler::handle(exception));
     }
 }
 
@@ -134,8 +123,8 @@ void ToDosController::deleteToDo(const HttpRequestPtr& req, std::function<void(c
         resp->setStatusCode(k200OK);
         callback(resp);
     }
-    catch (const std::exception& e)
+    catch (const std::exception& exception)
     {
-        callback(createInternalServerErrorResponse(e.what()));
+        callback(ExceptionResponseHandler::handle(exception));
     }
 }