From: Masaya TARUI <tarui@...>
Date: 2010-11-22T04:22:21+09:00
Subject: [ruby-core:33276] Re: [Ruby 1.9-Bug#4070][Open] possible XMLRPC connectino leak

--0015175cb2eaeae35b0495950ed8
Content-Type: text/plain; charset=ISO-8859-1

Hi,

It is not client side's problem. it's server side's one.
now, XMLRPC::Server.new() uses Webrick with maxConnections=4.

It's very poor, so server should not allow keep-alive.
I think that below patch is better. Even if it is not best.

Index: lib/xmlrpc/server.rb
===================================================================
--- lib/xmlrpc/server.rb        (revision 29847)
+++ lib/xmlrpc/server.rb        (working copy)
@@ -635,6 +635,7 @@
     super(*a)
     require 'webrick'
     @server = WEBrick::HTTPServer.new(:Port => port, :BindAddress => host,
:MaxClients => maxConnections,
+                                      :HTTPVersion    =>
HTTPVersion.new("1.0"),
                                       :Logger => WEBrick::Log.new(stdlog))
     @server.mount("/", self)
   end

-- 
Masaya TARUI
No Tool,No Life.

--0015175cb2eaeae35b0495950ed8
Content-Type: text/html; charset=ISO-8859-1
Content-Transfer-Encoding: base64

SGksIDxicj48YnI+SXQgaXMgbm90IGNsaWVudCBzaWRlJiMzOTtzIHByb2JsZW0uIGl0JiMzOTtz
IHNlcnZlciBzaWRlJiMzOTtzIG9uZS48YnI+bm93LKA8Y29kZT5YTUxSUEM6OlNlcnZlcjwvY29k
ZT4ubmV3KCkgdXNlcyBXZWJyaWNrIHdpdGggbWF4Q29ubmVjdGlvbnM9NC48YnI+PGJyPkl0JiMz
OTtzIHZlcnkgcG9vciwgc28gc2VydmVyIHNob3VsZCBub3QgYWxsb3cga2VlcC1hbGl2ZS48YnI+
CkkgdGhpbmsgdGhhdCBiZWxvdyBwYXRjaCBpcyBiZXR0ZXIuIEV2ZW4gaWYgaXQgaXMgbm90IGJl
c3QuPGJyPjxicj5JbmRleDogbGliL3htbHJwYy9zZXJ2ZXIucmI8YnI+PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PTxicj4t
LS0gbGliL3htbHJwYy9zZXJ2ZXIucmKgoKCgoKCgIChyZXZpc2lvbiAyOTg0Nyk8YnI+CisrKyBs
aWIveG1scnBjL3NlcnZlci5yYqCgoKCgoKAgKHdvcmtpbmcgY29weSk8YnI+QEAgLTYzNSw2ICs2
MzUsNyBAQDxicj6goKCgIHN1cGVyKCphKTxicj6goKCgIHJlcXVpcmUgJiMzOTt3ZWJyaWNrJiMz
OTs8YnI+oKCgoCBAc2VydmVyID0gV0VCcmljazo6SFRUUFNlcnZlci5uZXcoOlBvcnQgPSZndDsg
cG9ydCwgOkJpbmRBZGRyZXNzID0mZ3Q7IGhvc3QsIDpNYXhDbGllbnRzID0mZ3Q7IG1heENvbm5l
Y3Rpb25zLDxicj4KK6CgoKCgoKCgoKCgoKCgoKCgoKCgoKCgoKCgoKCgoKCgoKCgoKAgOkhUVFBW
ZXJzaW9uoKCgID0mZ3Q7IEhUVFBWZXJzaW9uLm5ldygmcXVvdDsxLjAmcXVvdDspLDxicj6goKCg
oKCgoKCgoKCgoKCgoKCgoKCgoKCgoKCgoKCgoKCgoKCgoCA6TG9nZ2VyID0mZ3Q7IFdFQnJpY2s6
OkxvZy5uZXcoc3RkbG9nKSk8YnI+oKCgoCBAc2VydmVyLm1vdW50KCZxdW90Oy8mcXVvdDssIHNl
bGYpPGJyPgqgoCBlbmQ8YnI+PGJyPi0tIDxicj5NYXNheWEgVEFSVUk8YnI+Tm8gVG9vbCxObyBM
aWZlLjxicj48YnI+Cg==
--0015175cb2eaeae35b0495950ed8--