letsencrypt / boulder / 9893
66%

DEFAULT BRANCH: master
Ran
Jobs 2
Files 88
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 63.025%. Remained the same
9893

cron

travis-ci

web-flow 
Use Challtestsrv for HTTP-01 integration tests, add redirect tests (#3960)

To complete https://github.com/letsencrypt/boulder/issues/3956 the `challtestsrv` is updated such that its existing TLS-ALPN-01 challenge test server will serve HTTP-01 responses with a self-signed certificate when a non-TLS-ALPN-01 request arrives. This lets the TLS-ALPN-01 challenge server double as a HTTPS version of the HTTP challenge server. The `challtestsrv` now also supports adding/remove redirects that will be served to clients when requesting matching paths.

The existing chisel/chisel2 integration tests are updated to use the `challtestsrv` instead of starting their own standalone servers. This centralizes our mock challenge responses and lets us bind the `challtestsrv` to the VA's HTTP port in `startservers.py` without clashing ports later on.

New integration tests are added for HTTP-01 redirect scenarios using the updated `challtestserv`. These test cases cover:
* valid HTTP -> HTTP redirect
* valid HTTP -> HTTPS redirect
* Invalid HTTP -> non-HTTP/HTTPS port redirect
* Invalid HTTP-> non-HTTP/HTTPS protocol scheme redirect
* Invalid HTTP-> bare IP redirect
* Invalid HTTP redirect loop

The new integration tests shook out two fixes that were required for the legacy VA HTTP-01 code (afad22b) and one fix for the challtestsrv mock DNS (59b7d6d).

Resolves https://github.com/letsencrypt/boulder/issues/3956

10798 of 17133 relevant lines covered (63.02%)

1.4 hits per line

Jobs
ID Job ID Ran Files Coverage
11 9893.11 (RUN="coverage" CONTAINER="netaccess") 0
63.02
 Travis Job 9893.11
12 9893.12 (RUN="coverage" CONTAINER="netaccess") 0
63.02
 Travis Job 9893.12
Source Files on build 9893
Detailed source file information is not available for this build.