Pebble does not support kid header yet

pull/55/head
Richard Körber 2017-04-27 22:38:54 +02:00
parent 4772488896
commit ee3765e543
2 changed files with 6 additions and 2 deletions

View File

@ -182,7 +182,10 @@ public class DefaultConnection implements Connection {
jws.getHeaders().setObjectHeaderValue("nonce", Base64Url.encode(session.getNonce()));
jws.getHeaders().setObjectHeaderValue("url", url);
if (session.getKeyIdentifier() != null) {
jws.getHeaders().setObjectHeaderValue("kid", session.getKeyIdentifier());
// TODO PEBBLE: cannot process "kid" yet, send "jwk" instead
// https://github.com/letsencrypt/pebble/issues/23
// jws.getHeaders().setObjectHeaderValue("kid", session.getKeyIdentifier());
jws.getHeaders().setJwkHeaderValue("jwk", jwk);
} else {
jws.getHeaders().setJwkHeaderValue("jwk", jwk);
}

View File

@ -615,7 +615,8 @@ public class DefaultConnectionTest {
expectedHeader.append("\"kid\":\"").append(keyIdentifier).append('"');
expectedHeader.append('}');
assertThat(header, sameJSONAs(expectedHeader.toString()));
// TODO PEBBLE: cannot process "kid" yet, send "jwk" instead
// assertThat(header, sameJSONAs(expectedHeader.toString()));
assertThat(claims, sameJSONAs("{\"foo\":123,\"bar\":\"a-string\"}"));
assertThat(signature, not(isEmptyOrNullString()));