Skip to content

Commit 172d704

Browse files
authored
Merge pull request #9 from coding-cpp/fix/3
namespace helpers->client
2 parents 40dd566 + adb1d69 commit 172d704

File tree

3 files changed

+19
-13
lines changed

3 files changed

+19
-13
lines changed

include/mochios/helpers/client.h

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,14 +15,18 @@ namespace mochios {
1515

1616
namespace helpers {
1717

18+
namespace client {
19+
1820
std::pair<std::string, std::string>
1921
buildRequest(mochios::messages::Request &request);
2022

2123
void buildResponse(mochios::messages::Response &res,
2224
std::stringstream &response);
2325

2426
mochios::messages::Response send(mochios::messages::Request &request,
25-
const int &socket);
27+
const int &socket);
28+
29+
} // namespace client
2630

2731
} // namespace helpers
2832

src/client/client.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -110,7 +110,7 @@ mochios::Client::sendHelper(mochios::messages::Request &request) {
110110
}
111111
this->connect();
112112
mochios::messages::Response res =
113-
mochios::helpers::send(request, this->socket);
113+
mochios::helpers::client::send(request, this->socket);
114114
for (const std::function<void(mochios::messages::Response &)>
115115
&responseInterceptor :
116116
this->internalInterceptors.response.interceptors) {

src/helpers/client.cpp

Lines changed: 13 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
#include <mochios/helpers/client.h>
22

33
std::pair<std::string, std::string>
4-
mochios::helpers::buildRequest(mochios::messages::Request &request) {
4+
mochios::helpers::client::buildRequest(mochios::messages::Request &request) {
55
std::ostringstream oss;
66
oss << request.method << " " << request.path;
77
if (request.queries.size() > 0) {
@@ -35,8 +35,8 @@ mochios::helpers::buildRequest(mochios::messages::Request &request) {
3535
return std::make_pair(oss.str(), jsonifiedBody);
3636
}
3737

38-
void mochios::helpers::buildResponse(mochios::messages::Response &res,
39-
std::stringstream &response) {
38+
void mochios::helpers::client::buildResponse(mochios::messages::Response &res,
39+
std::stringstream &response) {
4040
std::string line;
4141

4242
// Headers
@@ -69,23 +69,25 @@ void mochios::helpers::buildResponse(mochios::messages::Response &res,
6969
}
7070

7171
mochios::messages::Response
72-
mochios::helpers::send(mochios::messages::Request &request, const int &socket) {
72+
mochios::helpers::client::send(mochios::messages::Request &request,
73+
const int &socket) {
7374
std::pair<std::string, std::string> requestString =
74-
mochios::helpers::buildRequest(request);
75+
mochios::helpers::client::buildRequest(request);
7576
if (brewtils::sys::send(socket, requestString.first.c_str(),
7677
requestString.first.size(), 0) < 0) {
77-
logger::error("Error sending request headers",
78-
"mochios::messages::Response "
79-
"mochios::helpers::send(mochios::messages::Request &request, "
80-
"const mochios::enums::method &method, const int &socket)");
78+
logger::error(
79+
"Error sending request headers",
80+
"mochios::messages::Response "
81+
"mochios::helpers::client::send(mochios::messages::Request &request, "
82+
"const mochios::enums::method &method, const int &socket)");
8183
}
8284
if (requestString.second.size() > 0) {
8385
if (brewtils::sys::send(socket, requestString.second.c_str(),
8486
requestString.second.size(), 0) < 0) {
8587
logger::error(
8688
"Error sending request body",
8789
"mochios::messages::Response "
88-
"mochios::helpers::send(mochios::messages::Request &request, "
90+
"mochios::helpers::client::send(mochios::messages::Request &request, "
8991
"const mochios::enums::method &method, const int &socket)");
9092
}
9193
}
@@ -108,6 +110,6 @@ mochios::helpers::send(mochios::messages::Request &request, const int &socket) {
108110
res.statusText = parts[2];
109111
}
110112

111-
mochios::helpers::buildResponse(res, oss);
113+
mochios::helpers::client::buildResponse(res, oss);
112114
return res;
113115
}

0 commit comments

Comments
 (0)