5
66%
master: 66%

Ran 23 Apr 2018 01:29PM UTC

Files 85

Run time 4s

Badge

Embed ▾

Committed 23 Apr 2018 01:24PM UTC coverage: 62.404% (-0.002%) from 62.406%

Job # RUN="coverage"

Build Type

push

travis-ci

Committed by

cpu

Commit Message

Remove Timeout field from VA's http.Client. (#3661)

This field was set to singleDialTimeout, but the net/http library treats
it as covering all of dial, write headers, and read headers and body.
Since http01Dialer also uses singleDialTimeout, there's a race between
http01Dialer and net/http to see who will time out first. The result is
that sometimes we give "Timeout after connect" when the error really
should be "Timeout during connect." This issue also inhibits IPv6 to
IPv4 fallback, and tickles a data race that was causing a rare panic in
VA: https://github.com/letsencrypt/boulder/issues/3109.

After this change, the overall HTTP request will get the full deadline
allowed by the RPC context. The dialer will continue to use
singleDialTimeout for each of its two possible dial attempts.

Coverage Stats

9959 of 15959 relevant lines covered (62.4%)

0.69 hits per line

letsencrypt / boulder / 9041 / 5
66%
master: 66%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 9041.5 (RUN="coverage")

letsencrypt / boulder / 9041 / 5 66% master: 66%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 9041.5 (RUN="coverage")

letsencrypt / boulder / 9041 / 5
66%
master: 66%

README BADGES
x