[XrdHttp] Avoid sending empty header in the response as this conflict…

…s with the reported content-length
xrootd · Mar 21, 2023 · 3944814 · 3944814
1 parent a40e187
commit 3944814
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/src/XrdHttp/XrdHttpProtocol.cc b/src/XrdHttp/XrdHttpProtocol.cc
@@ -1529,7 +1529,7 @@ int XrdHttpProtocol::StartSimpleResp(int code, const char *desc, const char *hea
   if ((bodylen >= 0) && (code != 100))
     ss << "Content-Length: " << bodylen << crlf;
 
-  if (header_to_add)
+  if (header_to_add && (header_to_add[0] != '\0'))
     ss << header_to_add << crlf;
 
   ss << crlf;
@@ -1548,13 +1548,13 @@ int XrdHttpProtocol::StartSimpleResp(int code, const char *desc, const char *hea
 
 int XrdHttpProtocol::StartChunkedResp(int code, const char *desc, const char *header_to_add, bool keepalive) {
   const std::string crlf = "\r\n";
-
   std::stringstream ss;
-  if (header_to_add) {
+
+  if (header_to_add && (header_to_add[0] != '\0')) {
     ss << header_to_add << crlf;
   }
-  ss << "Transfer-Encoding: chunked";
 
+  ss << "Transfer-Encoding: chunked";
   TRACEI(RSP, "Starting chunked response");
   return StartSimpleResp(code, desc, ss.str().c_str(), -1, keepalive);
 }