8722
66%

Ran 06 Mar 2018 08:45PM UTC

Jobs 1

Files 84

Run time 279min

Badge

Embed ▾

pending completion

Build # 8722

Build Type

push

travis-ci

Committed by

rolandshoemaker

Commit Message

Make POSTing KeyAuthorization optional, V2 don't echo it. (#3526)

This commit updates the RA to make the notion of submitting
a KeyAuthorization value as part of the ra.UpdateAuthorization call
optional. If set, the value is enforced against expected and an error is
returned if the provided authorization isn't correct. If it isn't set
the RA populates the field with the computed authorization for the VA to
enforce against the value it sees in challenges. This retains the legacy
behaviour of the V1 API. The V2 API will never unmarshal a provided
key authorization.

The ACMEv2/WFEv2 prepChallengeForDisplay function is updated to strip
the ProvidedKeyAuthorization field before sending the challenge object
back to a client. ACMEv1/WFEv1 continue to return the KeyAuthorization
in challenges to avoid breaking clients that are relying on this legacy
behaviour.

For deployability ease this commit retains the name of the
core.Challenge.ProvidedKeyAuthorization field even though it should
be called core.Challenge.ComputedKeyAuthorization now that it isn't
set based on the client's provided key authz. This will be easier as
a follow-up change.

Resolves #3514

Coverage Stats

9154 of 15561 relevant lines covered (58.83%)

0.65 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
6	8722.6 (RUN="coverage")	06 Mar 2018 08:45PM UTC	0	58.83	Travis Job 8722.6

Source Files on build 8722

Detailed source file information is not available for this build.

letsencrypt / boulder / 8722
66%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 8722

letsencrypt / boulder / 8722 66%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 8722

letsencrypt / boulder / 8722
66%

README BADGES
x