[Patches] [ python-Patches-1627441 ] Fix for #1601399 (urllib2 does not close sockets properly)

[SourceForge.net]

Patches item #1627441, was opened at 2007-01-03 23:46
Message generated for change (Comment added) made by gbrandl
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=1627441&group_id=5470

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Library (Lib)
Group: Python 2.6
>Status: Closed
>Resolution: Accepted
Priority: 5
Private: No
Submitted By: John J Lee (jjlee)
Assigned to: Nobody/Anonymous (nobody)
Summary: Fix for #1601399 (urllib2 does not close sockets properly)

Initial Comment:
Fix for #1601399

Definitely a backport candidate.


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

>Comment By: Georg Brandl (gbrandl)
Date: 2007-01-21 10:35

Message:
Logged In: YES 
user_id=849994
Originator: NO

Committed as rev. 53511, 53512 (2.5).

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

Comment By: Mark Roberts (mark-roberts)
Date: 2007-01-21 05:26

Message:
Logged In: YES 
user_id=1591633
Originator: NO

I'd say it looks good.  Now lets see if we can get someone to apply it for
us.  Thanks for the adding the tests!

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

Comment By: John J Lee (jjlee)
Date: 2007-01-21 00:08

Message:
Logged In: YES 
user_id=261020
Originator: YES

Added tests.
File Added: urllib2_close_socket_v2.patch

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

Comment By: Mark Roberts (mark-roberts)
Date: 2007-01-20 02:44

Message:
Logged In: YES 
user_id=1591633
Originator: NO

Patch looks good to me, and the tests still pass.  If it matters, I would
like to see a test case presented in the patch as well.

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

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