Handle EAGAIN for windows users. Fixes #12

drbrain · Feb 6, 2012 · 232f975 · 232f975
1 parent 5f4a945
commit 232f975
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/History.txt b/History.txt
@@ -4,6 +4,10 @@
   * The allowed SSL version may now be set via #ssl_version.
     Issue #16 by astera
 
+* Bug fixes
+  * The ruby 1.8 speed monkeypatch now handles EAGAIN for windows users.
+    Issue #12 by Alwyn Schoeman
+
 === 2.4.1 / 2012-02-03
 
 * Bug fixes

diff --git a/lib/net/http/faster.rb b/lib/net/http/faster.rb
@@ -13,7 +13,7 @@ def rbuf_fill
     if @io.respond_to? :read_nonblock then
       begin
         @rbuf << @io.read_nonblock(65536)
-      rescue Errno::EWOULDBLOCK => e
+      rescue Errno::EWOULDBLOCK, Errno::EAGAIN => e
         retry if IO.select [@io], nil, nil, @read_timeout
         raise Timeout::Error, e.message
       end