2
91%
master: 91%

Ran 17 May 2017 07:42PM UTC

Files 17

Run time 1s

Badge

Embed ▾

Committed 17 May 2017 07:38PM UTC coverage: 90.904% (-0.2%) from 91.107%

Job # 1350.2

Build Type

push

travis-ci

Committed by

tylertreat

Commit Message

[FIXED] Authorization Timeout and TLS

When TLS and authorization is enabled, the authorization timeout can
fire during the TLS handshake, causing the server to write the
authorization timeout error string into the client socket, injecting
what becomes bad data into the TLS handshake. This creates misleading
errors on the client such as tls: oversized record received with length
21024.

This moves the authorization timeout scheduling to after the TLS
handshake to avoid the race. This should be safe since TLS has its own
handshake timeout. Added a unit test that fails with the old behavior
and passes with the new. LMK if you can think of a better way to test
this.

Fixes #432

Run Details

4917 of 5409 relevant lines covered (90.9%)

2854.96 hits per line

nats-io / gnatsd / 1350 / 2
91%
master: 91%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 1350.2

nats-io / gnatsd / 1350 / 2 91% master: 91%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 1350.2

nats-io / gnatsd / 1350 / 2
91%
master: 91%

README BADGES
x