From 0f9c65e489a7b59f76ccbf2ca6e76ab0a1012932 Mon Sep 17 00:00:00 2001
From: Karen Arutyunov <karen@codesynthesis.com>
Date: Tue, 1 Mar 2016 19:56:02 +0300
Subject: Fix web server responding with internal server error on POST request
 with empty body

---
 web/apache/request.ixx | 35 ++++++++++++++++++++++-------------
 1 file changed, 22 insertions(+), 13 deletions(-)

(limited to 'web/apache/request.ixx')

diff --git a/web/apache/request.ixx b/web/apache/request.ixx
index b3528d7..8a3b32b 100644
--- a/web/apache/request.ixx
+++ b/web/apache/request.ixx
@@ -2,7 +2,7 @@
 // copyright : Copyright (c) 2014-2016 Code Synthesis Ltd
 // license   : MIT; see accompanying LICENSE file
 
-#include <strings.h> // strcasecmp()
+#include <strings.h> // strncasecmp()
 
 #include <iomanip>
 #include <sstream>
@@ -53,22 +53,31 @@ namespace web
       if (!form_data_)
       {
         form_data_.reset (new std::string ());
-        const char* ct (apr_table_get (rec_->headers_in, "Content-Type"));
 
-        if (ct &&
-            strncasecmp ("application/x-www-form-urlencoded", ct, 33) == 0)
+        if (rec_->method_number == M_POST)
         {
-          std::istream& istr (content ());
-          std::getline (istr, *form_data_);
+          const char* ct (apr_table_get (rec_->headers_in, "Content-Type"));
 
-          // Make request data still be available.
-          //
-          std::unique_ptr<std::streambuf> in_buf (
-            new std::stringbuf (*form_data_));
+          if (ct != nullptr &&
+              strncasecmp ("application/x-www-form-urlencoded", ct, 33) == 0)
+          {
+            std::istream& istr (content ());
+
+            // Do not throw when eofbit is set (end of stream reached), and
+            // when failbit is set (getline() failed to extract any character).
+            //
+            istr.exceptions (std::ios::badbit);
+            std::getline (istr, *form_data_);
 
-          in_.reset (new std::istream (in_buf.get ()));
-          in_buf_ = std::move (in_buf);
-          in_->exceptions (std::ios::failbit | std::ios::badbit);
+            // Make this data the content of the input stream.
+            //
+            std::unique_ptr<std::streambuf> in_buf (
+              new std::stringbuf (*form_data_));
+
+            in_.reset (new std::istream (in_buf.get ()));
+            in_buf_ = std::move (in_buf);
+            in_->exceptions (std::ios::failbit | std::ios::badbit);
+          }
         }
       }
 
-- 
cgit v1.1