Remove Server header from responses
The HTTP Server header allows potential attackers to fingerprint the BMC
much easier than they could otherwise, as the bmc essentially reports
its name to requests.
From section 7.4.2 of RFC7231:
"An origin server MAY generate a Server field in its responses."
This patchset moves bmcwebs position that it will not publish the server
field, as it does not contain useful data for the client.
It should be noted, it looks like OpenSSL was using the server name for
its connection ID. It's not clear this is correct, or desired, but I've
inlined the old value (to avoid changing behavior). Also, it was
missing a return code check, so I added it.
Tested:
Will verify in the webui (TBD)
Signed-off-by: Ed Tanous <ed@tanous.net>
Change-Id: Ieee6f15d8299e76517952514ff196008a563b63c
diff --git a/http/http_server.h b/http/http_server.h
index 92971fb..c26e9e0 100644
--- a/http/http_server.h
+++ b/http/http_server.h
@@ -104,7 +104,7 @@
};
timer.async_wait(timerHandler);
- BMCWEB_LOG_INFO << serverName << " server is running, local endpoint "
+ BMCWEB_LOG_INFO << "bmcweb server is running, local endpoint "
<< acceptor->local_endpoint();
startAsyncWaitForSignal();
doAccept();
@@ -185,7 +185,7 @@
{
adaptorTemp = Adaptor(*ioService, *adaptorCtx);
auto p = std::make_shared<Connection<Adaptor, Handler>>(
- handler, serverName, getCachedDateStr, timerQueue,
+ handler, getCachedDateStr, timerQueue,
std::move(adaptorTemp.value()));
acceptor->async_accept(p->socket().next_layer(),
@@ -203,7 +203,7 @@
{
adaptorTemp = Adaptor(*ioService);
auto p = std::make_shared<Connection<Adaptor, Handler>>(
- handler, serverName, getCachedDateStr, timerQueue,
+ handler, getCachedDateStr, timerQueue,
std::move(adaptorTemp.value()));
acceptor->async_accept(
@@ -229,7 +229,6 @@
std::string dateStr;
Handler* handler;
- std::string serverName = "bmcweb";
std::function<void(const boost::system::error_code& ec)> timerHandler;