[Patches] [ python-Patches-415111 ] SocketServer doesn't close connection

noreply@sourceforge.net noreply@sourceforge.net
Tue, 10 Apr 2001 17:35:19 -0700


Patches item #415111, was updated on 2001-04-10 04:30
You can respond by visiting: 
http://sourceforge.net/tracker/?func=detail&atid=305470&aid=415111&group_id=5470

Category: library
Group: None
Status: Open
Priority: 6
Submitted By: Ka-Ping Yee (ping)
Assigned to: Nobody/Anonymous (nobody)
Summary: SocketServer doesn't close connection

Initial Comment:
SocketServer.StreamRequestHandler

    The setup() method here duplicates the socket twice
(once to
    make a read-only file, once to make a write-only
file).  That
    yields three descriptors, but finish() closes only
two.  This
    causes my browser to hang indefinitely waiting for
the socket
    to close when SimpleHTTPServer is used to deliver a
small page.

    This patch adds self.connection.close() to setup()
so that
    there are just two descriptors to worry about.


----------------------------------------------------------------------

>Comment By: Ka-Ping Yee (ping)
Date: 2001-04-10 17:35

Message:
Logged In: YES 
user_id=45338

Here's an updated patch that implements Guido's suggestion
to add a close_request method.

----------------------------------------------------------------------

Comment By: Guido van Rossum (gvanrossum)
Date: 2001-04-10 14:16

Message:
Logged In: YES 
user_id=6380

We're working through this on python-dev.

It's not so simple.

----------------------------------------------------------------------

You can respond by visiting: 
http://sourceforge.net/tracker/?func=detail&atid=305470&aid=415111&group_id=5470