ska-sa / aiokatcp / 98

96%

master: 95%

Pull #12

travis-ci

Fix endless loop of "socket.send() raised except"

If a client is receiving async informs (e.g. sensors) and disconnects,
the server won't notice that the underlying transport has closed and
would keep attempting to send informs, with the above message being
logged each time. It seems that asyncio assumes one regularly drains the
writer (and raises the ConnectionResetError from `drain`), but there is
no good place to put that (we don't want to block the server backend if
one client is slow, although that's a whole 'nother kettle of fish).

Fixed by manually checking for a closed stream. One now gets a single
instance of
```
WARNING:aiokatcp.connection:Connection closed before message could be sent: Connection lost [127.0.0.1:54100]
```
which I think might be due to TCP socket limitations, where you can't
tell that a socket has died until you try to write to it.

Closes #11.

2 of 2 new or added lines in 1 file covered. (100.0%)

2507 of 2601 relevant lines covered (96.39%)

1.93 hits per line