diff --git a/src/HttpController.cpp b/src/HttpController.cpp index 8332370..08b375a 100644 --- a/src/HttpController.cpp +++ b/src/HttpController.cpp @@ -70,7 +70,7 @@ void HttpController::uploadObject(const drogon::HttpRequestPtr &req, std::function &&callback) { auto server = Server::getInstance(); if (server && server->put_handler_) { - server->put_handler_->handle_put_object(req, std::move(callback)); + server->put_handler_->handle_upload_object(req, std::move(callback)); } else { auto resp = drogon::HttpResponse::newHttpResponse(); resp->setStatusCode(drogon::k500InternalServerError); diff --git a/src/put_handler.cpp b/src/put_handler.cpp index 9802f34..93d288f 100644 --- a/src/put_handler.cpp +++ b/src/put_handler.cpp @@ -26,7 +26,7 @@ namespace simple_object_storage { PutHandler::PutHandler(Server& server) : server_(server) {} -void PutHandler::handle_put_object(const drogon::HttpRequestPtr& req, std::function&& callback) { +void PutHandler::handle_upload_object(const drogon::HttpRequestPtr& req, std::function&& callback) { // Check all request parameters first before processing any data auto resp = drogon::HttpResponse::newHttpResponse(); @@ -36,16 +36,6 @@ void PutHandler::handle_put_object(const drogon::HttpRequestPtr& req, std::funct return; } - // 1. Check we're in the /upload path - if (req->getPath() != "/upload") { - resp->setStatusCode(drogon::k404NotFound); - nlohmann::json response = {{"result", "error"}, {"error", "Not found - put requests must be to /upload"}}; - resp->setBody(response.dump()); - resp->setContentTypeCode(drogon::CT_APPLICATION_JSON); - callback(resp); - return; - } - // Check content length first auto contentLengthHeader = req->getHeader("content-length"); if (!contentLengthHeader.empty()) { diff --git a/src/put_handler.hpp b/src/put_handler.hpp index 102018f..33b1464 100644 --- a/src/put_handler.hpp +++ b/src/put_handler.hpp @@ -11,7 +11,7 @@ namespace simple_object_storage { class PutHandler { public: PutHandler(Server& server); - void handle_put_object(const drogon::HttpRequestPtr& req, std::function&& callback); + void handle_upload_object(const drogon::HttpRequestPtr& req, std::function&& callback); private: Server& server_;