[issue1767370] Make xmlrpc use HTTP/1.1 and keepalive

[STINNER Victor]

Changes by STINNER Victor :


--
status: open -> closed

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue1767370] Make xmlrpc use HTTP/1.1 and keepalive

[Kristján Valur Jónsson]

Kristján Valur Jónsson  added the comment:

Yes, it is indeed a duplicate.

--
nosy: +krisvale
resolution:  -> duplicate

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue1767370] Make xmlrpc use HTTP/1.1 and keepalive

[STINNER Victor]

STINNER Victor  added the comment:

I think that this issue is a duplicate of #6267 which is already commited to
Python trunk. #6267 is a superset of #6099.

See also #2076 (another duplicate of #6267 ?).

--
nosy: +haypo

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue1767370] Make xmlrpc use HTTP/1.1 and keepalive

[williambr]

williambr  added the comment:

Hi.

I am very intereasted about this patch... specially about the keep-alive
part...
I have a proposition... wouldn't be fair enough to have a separated
Transport just for keep-alive connections?

I could wirte one if no one is against it...

--
nosy: +williambr

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue1767370] Make xmlrpc use HTTP/1.1 and keepalive

[Ionut Turturica]

Ionut Turturica <[EMAIL PROTECTED]> added the comment:

Note that win32 Python's socket module doesn't have a MSG_DONTWAIT
constant defined. So the following code will fail on windows machines.

+self.__connection.sock.recv(1,
+socket.MSG_PEEK | 
+socket.MSG_DONTWAIT)

Good job with this patch. It would've been interesting to have a flag to
switch to old http1.0 and see the differences in terms of performance.

Ionut

--
nosy: +jonozzz
type:  -> behavior

___
Python tracker <[EMAIL PROTECTED]>

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue1767370] Make xmlrpc use HTTP/1.1 and keepalive

[Martin v. Löwis]

Martin v. Löwis <[EMAIL PROTECTED]> added the comment:

If this is under the Google agreement, then it's fine (I think). It's
just that we can't accept anonymous contributions (even if made through
a known middleman).

___
Python tracker <[EMAIL PROTECTED]>

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue1767370] Make xmlrpc use HTTP/1.1 and keepalive

[Martín Conte Mac Donell]

Martín Conte Mac Donell <[EMAIL PROTECTED]> added the comment:

I've signed and faxed the form. Just in case.

Martin.

___
Python tracker <[EMAIL PROTECTED]>

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue1767370] Make xmlrpc use HTTP/1.1 and keepalive

[Donovan Baarda]

Donovan Baarda <[EMAIL PROTECTED]> added the comment:

On Tue, July 22, 2008 05:21, Martin v. Löwis wrote:
>
> Martin v. Löwis <[EMAIL PROTECTED]> added the comment:
>
> We would need the copyright holder of the patch to submit a contributor
> form. Would that be possible?

he works for Google ([EMAIL PROTECTED]), and so do I ([EMAIL PROTECTED])... I
think Google has some sort of company wide contributor agreement, and I
faintly recall signing something before I started working at Google...

krabbe also has a more recent version of this patch that we both keep
failing to submit..

Please let me know if he still needs to sign something, and I'll try to
get him to submit his most recent version of this...

> --
> nosy: +loewis
>
> ___
> Python tracker <[EMAIL PROTECTED]>
> 
> ___
>

___
Python tracker <[EMAIL PROTECTED]>

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue1767370] Make xmlrpc use HTTP/1.1 and keepalive

[Martin v. Löwis]

Martin v. Löwis <[EMAIL PROTECTED]> added the comment:

We would need the copyright holder of the patch to submit a contributor
form. Would that be possible?

--
nosy: +loewis

___
Python tracker <[EMAIL PROTECTED]>

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue1767370] Make xmlrpc use HTTP/1.1 and keepalive

[Mark Harrison]

Mark Harrison <[EMAIL PROTECTED]> added the comment:

There's a one-line change necessary in BaseHTTPServer.py.
s/socketserver/SocketServer/ on this line:

+socketserver.StreamRequestHandler.__init__(self, request,
client_address, parent)

+SocketServer.StreamRequestHandler.__init__(self, request,
client_address, parent)

Other than that it seems to work well.  The client side xmlrpc
code even does the right thing with an apache-based server.

I copied just the patched files to a standalone directory and
it works for Python 2.4 as well, if sys.path picks up that
directory first.

--
nosy: +marhar

___
Python tracker <[EMAIL PROTECTED]>

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue1767370] Make xmlrpc use HTTP/1.1 and keepalive

[Donovan Baarda]

Donovan Baarda <[EMAIL PROTECTED]> added the comment:

One more time... this time after adding correct email addresses to my 
existing account...

Martín Conte Mac Donell wrote:
> Martín Conte Mac Donell <[EMAIL PROTECTED]> added the comment:
> 
> I made this patch works against trunk, also i'v fixed some typos.

Sorry for not responding to this earlier...

I didn't write this patch, but submitted it on behalf of the original
author (Cc'ed on this email). I spoke to him about getting this patch
updated, and he said that he has made some more improvements and fixes
since I submitted it.

I will try to get him to produce a new patch, but he (and I) are both
very busy on other things, so I suspect you guys will be able to polish
  up what is already there before we manage to pull something together...

I will give him another poke tomorrow...

> --
> nosy: +Reflejo
> versions: +Python 2.5
> Added file: http://bugs.python.org/file10313/xmlrpc-keepalive.diff
> 
> _
> Tracker <[EMAIL PROTECTED]>
> 
> _

_
Tracker <[EMAIL PROTECTED]>

_
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue1767370] Make xmlrpc use HTTP/1.1 and keepalive

[Skip Montanaro]

Skip Montanaro <[EMAIL PROTECTED]> added the comment:

I'm not going to pretend I understand the changes.  I do notice
that test_docxmlrpc hangs hard on my Mac (doesn't even respond to
Ctl-C).  DocXMLRPCServer subclasses from SimpleXMLRPCServer, so I
suspect it needs some attention.  I tried the obvious replacement
of self.wfile.* with the analogs from SimpleXMLRPCServer but that
didn't help.  Also, the API for the xmlrpclib.Transport class's
send_request method changed and parse_response disappeared completely
even though the docstring for the Transport class explicitly mentions
subclassing it.

_
Tracker <[EMAIL PROTECTED]>

_
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue1767370] Make xmlrpc use HTTP/1.1 and keepalive

[Martín Conte Mac Donell]

Martín Conte Mac Donell <[EMAIL PROTECTED]> added the comment:

I made this patch works against trunk, also i'v fixed some typos.

--
nosy: +Reflejo
versions: +Python 2.5
Added file: http://bugs.python.org/file10313/xmlrpc-keepalive.diff

_
Tracker <[EMAIL PROTECTED]>

_
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue1767370] Make xmlrpc use HTTP/1.1 and keepalive

[Ralf Schmitt]

Changes by Ralf Schmitt <[EMAIL PROTECTED]>:


--
nosy: +schmir

_
Tracker <[EMAIL PROTECTED]>

_
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue1767370] Make xmlrpc use HTTP/1.1 and keepalive

[Raymond Hettinger]

Raymond Hettinger <[EMAIL PROTECTED]> added the comment:

Also, please post the patch in uncompressed form.

--
nosy: +rhettinger

_
Tracker <[EMAIL PROTECTED]>

_
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue1767370] Make xmlrpc use HTTP/1.1 and keepalive

[Skip Montanaro]

Skip Montanaro <[EMAIL PROTECTED]> added the comment:

Donovan, can you recreate this patch using the current Subversion trunk?
 The patch program complains:

 malformed patch at line 125: Index:
trunk.2/Lib/SimpleXMLRPCServer.py

Thanks,

Skip

--
nosy: +skip.montanaro

_
Tracker <[EMAIL PROTECTED]>

_
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com