commit 33c6b58026e5dd3bc600d152a07b95f863bfa1a7
author Ed Tanous <edtanous@google.com> Tue Feb 14 15:05:48 2023 -0800
committer Ed Tanous <ed@tanous.net> Sat Mar 11 00:50:32 2023 +0000
tree 17d8803445b5310f48eba160c2fc488a70fcd215
parent 98fe740b59a96fd4d4232655f820da168129f090

Remove body member from Request

Per cpp core guidelines, these should be methods.

Tested: on last patchset of the series.

Signed-off-by: Ed Tanous <edtanous@google.com>
Change-Id: Ib16479db9d2b68da68e7ad6e825c7e205c64f1de

include/ibm/management_console_rest.hpp

diff --git a/include/ibm/management_console_rest.hpp b/include/ibm/management_console_rest.hpp
index c56b390..e1b7f83 100644
--- a/include/ibm/management_console_rest.hpp
+++ b/include/ibm/management_console_rest.hpp
@@ -117,7 +117,7 @@
     BMCWEB_LOG_DEBUG << "saveAreaDirSize: " << saveAreaDirSize;
 
     // Get the file size getting uploaded
-    const std::string& data = req.body;
+    const std::string& data = req.body();
     BMCWEB_LOG_DEBUG << "data length: " << data.length();
 
     if (data.length() < minSaveareaFileSize)

include/image_upload.hpp

diff --git a/include/image_upload.hpp b/include/image_upload.hpp
index ab2901e..18a0c09 100644
--- a/include/image_upload.hpp
+++ b/include/image_upload.hpp
@@ -99,7 +99,7 @@
     BMCWEB_LOG_DEBUG << "Writing file to " << filepath;
     std::ofstream out(filepath, std::ofstream::out | std::ofstream::binary |
                                     std::ofstream::trunc);
-    out << req.body;
+    out << req.body();
     out.close();
     timeout.async_wait(timeoutHandler);
 }

include/login_routes.hpp

diff --git a/include/login_routes.hpp b/include/login_routes.hpp
index e3b8c19..00debb5 100644
--- a/include/login_routes.hpp
+++ b/include/login_routes.hpp
@@ -37,7 +37,8 @@
         // Check if auth was provided by a payload
         if (contentType.starts_with("application/json"))
         {
-            loginCredentials = nlohmann::json::parse(req.body, nullptr, false);
+            loginCredentials =
+                nlohmann::json::parse(req.body(), nullptr, false);
             if (loginCredentials.is_discarded())
             {
                 BMCWEB_LOG_DEBUG << "Bad json in request";

include/multipart_parser.hpp

diff --git a/include/multipart_parser.hpp b/include/multipart_parser.hpp
index 4824269..adeeccb 100644
--- a/include/multipart_parser.hpp
+++ b/include/multipart_parser.hpp
@@ -74,8 +74,8 @@
         lookbehind.resize(boundary.size() + 8);
         state = State::START;
 
-        const char* buffer = req.body.data();
-        size_t len = req.body.size();
+        const char* buffer = req.body().data();
+        size_t len = req.body().size();
         char cl = 0;
 
         for (size_t i = 0; i < len; i++)