Merge pull request microsoft#260 from johnpaultaken/dev3

ras0219-msft · web-flow · commit 006c09ff28c0 · 2016-11-03T17:53:14.000-07:00
ECONNRESET must be used instead of ECONNABORTED.
diff --git a/Release/src/http/listener/http_server_asio.cpp b/Release/src/http/listener/http_server_asio.cpp
@@ -249,7 +249,12 @@ void connection::handle_http_line(const boost::system::error_code& ec)
     if (ec)
     {
         // client closed connection
-        if (ec == boost::asio::error::eof || ec == boost::asio::error::operation_aborted)
+        if (
+            ec == boost::asio::error::eof ||                // peer has performed an orderly shutdown
+            ec == boost::asio::error::operation_aborted ||  // this can be removed. ECONNABORTED happens only for accept()
+            ec == boost::asio::error::connection_reset ||   // connection reset by peer
+            ec == boost::asio::error::timed_out             // connection timed out
+        )
         {
             finish_request_response();
         }

Original file line number	Diff line number	Diff line change
`@@ -249,7 +249,12 @@ void connection::handle_http_line(const boost::system::error_code& ec)`
`249`	`249`	`if (ec)`
`250`	`250`	`{`
`251`	`251`	`// client closed connection`
`252`		`- if (ec == boost::asio::error::eof \|\| ec == boost::asio::error::operation_aborted)`
	`252`	`+ if (`
	`253`	`+ ec == boost::asio::error::eof \|\| // peer has performed an orderly shutdown`
	`254`	`+ ec == boost::asio::error::operation_aborted \|\| // this can be removed. ECONNABORTED happens only for accept()`
	`255`	`+ ec == boost::asio::error::connection_reset \|\| // connection reset by peer`
	`256`	`+ ec == boost::asio::error::timed_out // connection timed out`
	`257`	`+ )`
`253`	`258`	`{`
`254`	`259`	`finish_request_response();`
`255`	`260`	`}`