9041
66%

Ran 23 Apr 2018 01:29PM UTC

Jobs 1

Files 85

Run time 5s

Badge

Embed ▾

pending completion

Build # 9041

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

Jobs

ID	Job ID	Ran	Files	Coverage
5	9041.5 (RUN="coverage")	23 Apr 2018 01:29PM UTC	0	62.4	Travis Job 9041.5

Source Files on build 9041

Detailed source file information is not available for this build.

letsencrypt / boulder / 9041
66%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 9041

letsencrypt / boulder / 9041 66%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 9041

letsencrypt / boulder / 9041
66%

README BADGES
x