Giampaolo Rodola' <billiej...@users.sourceforge.net> added the comment:

This time it's not easy as I see no way to distinguish whether the 
timeout exception gets raised by the command or the control socket, as 
makeport() method implementation deals with both:

<snippet>
        host = self.sock.getsockname()[0] 
        if self.af == socket.AF_INET:
            resp = self.sendport(host, port)  # socket.timeout can be 
raised here
        else:
            resp = self.sendeprt(host, port)   # ...here
        if self.timeout is not _GLOBAL_DEFAULT_TIMEOUT:
            sock.settimeout(self.timeout)    # or here
</snippet>

I think the best we can do is add a test which checks the timeout 
applied to the data socket resulting from a PASV/PORT request.
That doesn't cover this specific bug at all but it's something which is 
currently missing and that it would be nice to have.

----------

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue4814>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to