Be sure to cleanup and close bad client sockets

puma · Jul 24, 2012 · e13d9ba · e13d9ba
1 parent 44c8c1a
commit e13d9ba
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 0 deletions.
diff --git a/lib/puma/reactor.rb b/lib/puma/reactor.rb
@@ -36,6 +36,9 @@ def run
                 end
               # The client doesn't know HTTP well
               rescue HttpParserError => e
+                c.close
+                sockets.delete c
+
                 @events.parse_error @server, c.env, e
 
               rescue EOFError

diff --git a/lib/puma/server.rb b/lib/puma/server.rb
@@ -205,6 +205,7 @@ def run(background=true)
         begin
           process_now = client.eagerly_finish
         rescue HttpParserError => e
+          client.close
           @events.parse_error self, client.env, e
         rescue EOFError
           client.close