[issue2450] urllib2.Request constructor has no timeout parameter

2008-03-21 Thread John J Lee
New submission from John J Lee [EMAIL PROTECTED]: r55792 added timeout support to urllib2. A timeout parameter was added to urllib2.OpenerDirector.open(), but there is no corresponding Request constructor parameter. timeout is unique in that respect. Instead, OpenerDirector.open() sets

[issue2450] urllib2.Request constructor has no timeout parameter

2008-03-21 Thread Benjamin Peterson
Changes by Benjamin Peterson [EMAIL PROTECTED]: -- nosy: +facundobatista __ Tracker [EMAIL PROTECTED] http://bugs.python.org/issue2450 __ ___ Python-bugs-list mailing list

[issue2450] urllib2.Request constructor has no timeout parameter

2008-03-21 Thread Facundo Batista
Changes by Facundo Batista [EMAIL PROTECTED]: -- assignee: - facundobatista __ Tracker [EMAIL PROTECTED] http://bugs.python.org/issue2450 __ ___ Python-bugs-list mailing list

[issue2450] urllib2.Request constructor has no timeout parameter

2008-03-21 Thread Facundo Batista
Facundo Batista [EMAIL PROTECTED] added the comment: The argument timeout can not have the same semantics than data, see the following cases: - r = Request(url) - urlopen(r, data=foo) - # data is foo - r = Request(url, data=foo) - urlopen(r) - # data is foo - r = Request(url, data=foo) -

[issue2450] urllib2.Request constructor has no timeout parameter

2008-03-21 Thread John J Lee
John J Lee [EMAIL PROTECTED] added the comment: This should be solved by introducing a not set value other than None. __ Tracker [EMAIL PROTECTED] http://bugs.python.org/issue2450 __ ___

[issue2450] urllib2.Request constructor has no timeout parameter

2008-03-21 Thread Facundo Batista
Facundo Batista [EMAIL PROTECTED] added the comment: As I wrote in the other bug that you opened (#2451), introducing this will complicate the usage and semantics of what is already established and working. __ Tracker [EMAIL PROTECTED]

[issue2450] urllib2.Request constructor has no timeout parameter

2008-03-21 Thread John J Lee
John J Lee [EMAIL PROTECTED] added the comment: I don't buy the API complication argument. I might accept an argument that the timeout isn't really anything to do with the request, so I won't bother to continue with this bug report. __ Tracker [EMAIL PROTECTED]