From: Bryan Duxbury <bryanduxbury@apache.org>
Date: Fri, 15 Jan 2010 17:30:40 +0000 (+0000)
Subject: THRIFT-604: rb: nonblocking_server assumes that socket.read returns exactly one message
X-Git-Tag: 0.3.0~137
X-Git-Url: https://source.supwisdom.com/gerrit/gitweb?a=commitdiff_plain;h=8d41b643f9bb670f18404260fbe124c0cabb5360;p=common%2Fthrift.git

THRIFT-604: rb: nonblocking_server assumes that socket.read returns exactly one message

This patch causes every message currently in the buffer to be processed before going on.

git-svn-id: https://svn.apache.org/repos/asf/incubator/thrift/trunk@899725 13f79535-47bb-0310-9956-ffa450edef68
---

diff --git a/lib/rb/lib/thrift/server/nonblocking_server.rb b/lib/rb/lib/thrift/server/nonblocking_server.rb
index 5425f6de..9e1cfe6b 100644
--- a/lib/rb/lib/thrift/server/nonblocking_server.rb
+++ b/lib/rb/lib/thrift/server/nonblocking_server.rb
@@ -160,8 +160,7 @@ module Thrift
 
       def read_connection(fd)
         @buffers[fd] << fd.read(DEFAULT_BUFFER)
-        frame = slice_frame!(@buffers[fd])
-        if frame
+        while(frame = slice_frame!(@buffers[fd]))
           @logger.debug "#{self} is processing a frame"
           @worker_queue.push [:frame, fd, frame]
         end