Title: no address in the representation of asyncore dispatcher after connection established
Type: behavior Stage: patch review
Components: Library (Lib) Versions: Python 3.2, Python 3.3, Python 2.7
Status: closed Resolution: duplicate
Dependencies: Superseder:
Assigned To: Nosy List: giampaolo.rodola, josiahcarlson, stutzbach, xdegaye
Priority: normal Keywords: patch

Created on 2011-11-02 16:55 by xdegaye, last changed 2012-03-22 14:25 by giampaolo.rodola. This issue is now closed.

File name Uploaded Description Edit xdegaye, 2011-11-02 16:55
dispatcher_addr.diff xdegaye, 2011-11-02 16:56 review
Messages (2)
msg146849 - (view) Author: Xavier de Gaye (xdegaye) * (Python triager) Date: 2011-11-02 16:55
When an asyncore dispatcher initiates a tcp connection, its
representation lacks the peer address. The output of the attached
script '' gives on linux with Python 3.2:

call stack in handle_connect_event:
self.addr after connection: None

The attached patch fixes the problem. The patch includes a test case.
The following comment in the patch:

    # EWOULDBLOCK may also be returned by winsock when calling connect
    # while the connection attempt is in progress

refers to the following statement in

As a result, it is not recommended that applications use multiple
calls to connect to detect connection completion. If they do, they
must be prepared to handle WSAEINVAL and WSAEWOULDBLOCK error codes
the same way that they handle WSAEALREADY to ensure robust execution.

Note that two consecutive calls to
socket.getsockopt(socket.SOL_SOCKET, socket.SO_ERROR) are made when
handle_write_event() is called and self.connected is False: one in
handle_write_event and the following right away in
handle_connect_event. This seems useless.
msg156563 - (view) Author: Giampaolo Rodola' (giampaolo.rodola) * (Python committer) Date: 2012-03-22 14:25
Closing as duplicate of issue 13694.
Date User Action Args
2012-03-22 14:25:29giampaolo.rodolasetstatus: open -> closed
resolution: duplicate
messages: + msg156563

versions: + Python 2.7
2011-11-03 01:58:41pitrousetnosy: + josiahcarlson, giampaolo.rodola, stutzbach
stage: patch review

versions: + Python 3.3
2011-11-02 16:56:16xdegayesetfiles: + dispatcher_addr.diff
keywords: + patch
2011-11-02 16:55:39xdegayesetfiles: +
2011-11-02 16:55:01xdegayecreate