Login | Register For Free | Help
Search for: (Advanced)

Mailing List Archive: Python: Dev

API bug in new socket timeout interfaces

 

 

Python dev RSS feed   Index | Next | Previous | View Threaded


jjl at pobox

May 26, 2008, 1:38 PM

Post #1 of 2 (298 views)
Permalink
API bug in new socket timeout interfaces

I'm hoping to draw somebody's attention to this before it's too late.
This affects multiple stdlib modules:

http://bugs.python.org/issue2451

"""
The new timeout support in 2.6 makes use of new function
socket.create_connection(). socket.create_connection() provides no way to
disable timeouts, other than by relying on socket.getdefaulttimeout()
returning None. This is unfortunate, because it was the purpose of the
new timeout support to allow control of timeouts without reliance on
global state.
"""

The author of the patch that introduced this problem (Facundo Batista) has
admitted that the behaviour he implemented does not match the discussion
about the feature that took place on python-dev, but has not found time to
review my patch in the last two months. I fear this broken API will make
it into 2.6 (and 3.0?).

More could be done here re other issues introduced by the patch (I've
listed those that I noticed on issue 2451 -- including at least one other
API issue), but the patch attached to issue2451 at least resolves the most
clear-cut problem.

Thanks


John

_______________________________________________
Python-Dev mailing list
Python-Dev [at] python
http://mail.python.org/mailman/listinfo/python-dev
Unsubscribe: http://mail.python.org/mailman/options/python-dev/list-python-dev%40lists.gossamer-threads.com


facundobatista at gmail

May 26, 2008, 5:28 PM

Post #2 of 2 (280 views)
Permalink
Re: API bug in new socket timeout interfaces [In reply to]

2008/5/26 John J Lee <jjl [at] pobox>:

> The author of the patch that introduced this problem (Facundo Batista) has
> admitted that the behaviour he implemented does not match the discussion
> about the feature that took place on python-dev, but has not found time to
> review my patch in the last two months. I fear this broken API will make it
> into 2.6 (and 3.0?).

No, I'll review it in less than two weeks.

--
. Facundo

Blog: http://www.taniquetil.com.ar/plog/
PyAr: http://www.python.org/ar/
_______________________________________________
Python-Dev mailing list
Python-Dev [at] python
http://mail.python.org/mailman/listinfo/python-dev
Unsubscribe: http://mail.python.org/mailman/options/python-dev/list-python-dev%40lists.gossamer-threads.com

Python dev RSS feed   Index | Next | Previous | View Threaded
 
 


Interested in having your list archived? Contact Gossamer Threads
 
  Web Applications & Managed Hosting Powered by Gossamer Threads Inc.