Slow timeout when trying to connect to an invalid IP address

From #615

python -m pynetdicom echoscu [invalid IP] 11112 -d -ta 5

Should immediately fail due to non-existent IP, but doesn’t.

[scaramallion]

Fixed with #618

[scaramallion]

After writing all the below, the simplest simplest way is to probably just put None on DUL.to_user_queue when the connection fails so the blocking rsp = self.dul.receive_pdu(...) returns. Maybe see if there’s a way to get a better log message in that case.

---

The state machine for the start of association requests is

Sta1 + Evt1 -> AE-1 -> Sta4
Sta4 + Evt2 -> AE-2 -> Sta5

Where

Sta1: Idle
Sta4: Awaiting transport connection opening to complete (from local transport service)
Sta5: Awaiting A-ASSOCIATE-AC or A-ASSOCIATE-RJ PDU

Evt1: A-ASSOCIATE request received from local user)
Evt2: Transport connection confirmation from local transport service

So while we’re operating in Sta4, the ACSE timeout should be idle until Evt2 occurs and we reach Sta5. There seems to be a step missing in _negotiate_as_requestor() after sending the A-ASSOCIATE request primitive to the DUL in send_request() and prior to rsp = self.dul.receive_pdu(wait=True, timeout=self.acse_timeout) where we wait until the connection has been made.

I’d be inclined to put in a timeout check (based on network_timeout?) in between that waits on Sta5.

self.send_request()

while self.assoc.dul.state_machine.state != 'Sta5':
    # wait until timeout expires

evt.trigger(self.assoc, evt.EVT_REQUESTED, {})

It’s probably better to use a threading event there with the other end tied to the socket.connect() code so the ACSE only containues once the connection request is made or failed.

Alternatively you could just lock the thread into the socket.connect() code until timeout… but that might have weird effects when running an AE with multiple SCPs that needs to make association requests.

I really hate threading…