1
96%
master: 95%

Ran 28 Jan 2019 03:13PM UTC

Files 15

Run time 0s

Badge

Embed ▾

Committed 28 Jan 2019 03:08PM UTC coverage: 96.386%. First build

Job # 97.1

Build Type

push

travis-ci

Committed by

bmerry

Commit Message

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.

Run Details

2507 of 2601 relevant lines covered (96.39%)

0.96 hits per line

ska-sa / aiokatcp / 97 / 1
96%
master: 95%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 97.1

ska-sa / aiokatcp / 97 / 1 96% master: 95%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 97.1

ska-sa / aiokatcp / 97 / 1
96%
master: 95%

README BADGES
x