async-http 0.27.3 → 0.27.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 62159f76d28951cd1318ee83403215626a3e6a094fcccaec0f67ab4865f50ce4
-  data.tar.gz: de18a9f234cacc5943468cc9454e8bb8102c3dac38a81129b0dc9cebc1820368
+  metadata.gz: d6fe0971913f89e9ebd68515c2e93ae525757490c52d36ab1b28a6ecc6f1257f
+  data.tar.gz: aa2e52fada5c8e31e634d40f84eca6fe4f4fa604eb7eeb220e7b020a8fdeb4cf
 SHA512:
-  metadata.gz: dd79c554a941ba4e0d33221c642a003b90c8459faf3a8a8c025fdeb9f5ef03db9f30f4ff37ce3b26511447b5b903c96783f8109b4689ba417779dccc3e0b5b20
-  data.tar.gz: e807c841615de91a7d97c20a47c80c1c9e30246980d1ff627f4c88486d147d8108e1d1cd953928b0a02e350fbc95a88e45c053e5601ad5625bc62de431b40981
+  metadata.gz: 44a409a36d515ff781392703c8daa76984f02f56ff6860f86939fa8740adf940d9ac313a2b5436afc3939e4162c46fd56c7e19518071992b9637d3bac703cd92
+  data.tar.gz: b4b2869437271dda4f1dbf748c852fd6ac8d4f0e2e1aa3e3c495601a25e9fdf121562325c61436b2ca1de3a92551437e3403409a028a7373e7ab722f41838c72

data/lib/async/http/protocol/http11.rb

@@ -85,6 +85,7 @@ module Async
 					end
 				end
 				
+				# @return [Async::Wrapper] the underlying non-blocking IO.
 				def hijack
 					@persistent = false
 					
@@ -130,15 +131,22 @@ module Async
 				# Server loop.
 				def receive_requests(task: Task.current)
 					while request = next_request
-						if response = yield(request, self)
+						response = yield(request, self)
+						
+						return if @stream.closed?
+						
+						if response
 							write_response(self.version, response.status, response.headers, response.body)
-							request.finish
-							
-							# This ensures we yield at least once every iteration of the loop and allow other fibers to execute.
-							task.yield
 						else
-							break
+							# If the request failed to generate a response, it was an internal server error:
+							write_response(self.version, 500, {}, nil)
 						end
+						
+						# Gracefully finish reading the request body if it was not already done so.
+						request.finish
+						
+						# This ensures we yield at least once every iteration of the loop and allow other fibers to execute.
+						task.yield
 					end
 				end
 				

data/lib/async/http/protocol/http2.rb

@@ -235,7 +235,7 @@ module Async
 							stream.data("", end_stream: true)
 						end
 					else
-						stream.close(:internal_error) unless stream.state == :closed
+						stream.headers({':status' => '500'}, end_stream: true)
 					end
 				rescue
 					Async.logger.error(request) {$!}

data/lib/async/http/response.rb

@@ -64,6 +64,10 @@ module Async
 				status == 400
 			end
 			
+			def server_failure?
+				status == 500
+			end
+			
 			def self.[](status, headers = {}, body = [])
 				body = Body::Buffered.wrap(body)
 				

data/lib/async/http/version.rb

@@ -20,6 +20,6 @@
 
 module Async
 	module HTTP
-		VERSION = "0.27.3"
+		VERSION = "0.27.4"
 	end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: async-http
 version: !ruby/object:Gem::Version
-  version: 0.27.3
+  version: 0.27.4
 platform: ruby
 authors:
 - Samuel Williams
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-06-29 00:00:00.000000000 Z
+date: 2018-07-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: async