6
99%
master: 3%

Ran 08 Nov 2017 09:43PM UTC

Files 87

Run time 5s

Badge

Embed ▾

Committed 08 Nov 2017 09:25PM UTC coverage: 99.169% (-0.09%) from 99.263%

Job # jruby-head

Build Type

push

travis-ci

Committed by

sferik

Commit Message

[6-stable] Add TCP keepalive in Twitter::Streaming::Connection (#856)

* Add TCP keepalive in Twitter::Streaming::Connection

Without correct TCP keepalive setup, the Twitter streaming client is
prone to hang on long-lived connections. This is due to the nature of
the protocol: a single request is made, then the client simply waits for
data to read on a socket. Without keepalive, it may be indistinguishable
to the client whether the connection has failed or there is simply no
data to read.

These changes add the ability to set TCP keepalive providing the OS
supports it. They also provide a set of sensible defaults to the
keepalive for this implementation, while disabled by default.

* Add tests around keepalive setup

Run Details

2029 of 2046 relevant lines covered (99.17%)

33.32 hits per line

sferik / twitter / 1693 / 6
99%
master: 3%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 1693.6 (jruby-head)

sferik / twitter / 1693 / 6 99% master: 3%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 1693.6 (jruby-head)

sferik / twitter / 1693 / 6
99%
master: 3%

README BADGES
x