Refine when content is expected

Consider Content-Length and Transfer-Encoding headers when determining
whether to expect content. Don't expect content for DELETE requests and
don't handle the HTTP/2 connection preface pseudo-method PRI.

Fixes #2028.

Author: falbrechtskirchinger

httplib.h

@@ -5380,11 +5380,14 @@ write_multipart_ranges_data(Stream &strm, const Request &req, Response &res,
 }
 
 inline bool expect_content(const Request &req) {
-  if (req.method == "POST" || req.method == "PUT" || req.method == "PATCH" ||
-      req.method == "PRI" || req.method == "DELETE") {
+  if (req.method == "POST" || req.method == "PUT" || req.method == "PATCH") {
     return true;
   }
-  // TODO: check if Content-Length is set
+  if (req.has_header("Content-Length") &&
+      req.get_header_value_u64("Content-Length") > 0) {
+    return true;
+  }
+  if (is_chunked_transfer_encoding(req.headers)) { return true; }
   return false;
 }