no-acme #10

Fusionado
lastlink fusionados 13 commits de lastlink/bluecrypt-keypairs.js:no-acme en master 2019-05-11 22:32:00 +00:00
Colaborador
  • fixed merge issue
  • jwk to jwt & jws
* fixed merge issue * jwk to jwt & jws
coolaj86 aprobado estos cambios 2019-05-11 21:58:39 +00:00
coolaj86 dejó un comentario
Propietario

Approving so this can merge, but will you PR again to remove the debug value for the key and use the generated key instead?

Approving so this can merge, but will you PR again to remove the debug value for the key and use the generated key instead?
@ -52,3 +54,2 @@
kty: $('input[name="kty"]:checked').value
, namedCurve: $('input[name="ec-crv"]:checked').value
, modulusLength: $('input[name="rsa-len"]:checked').value
, namedCurve: $('input[name="ec-crv"]:checked').value
Propietario

these aren't aligned. I wouldn't stop a merge for that, just want you to notice.

these aren't aligned. I wouldn't stop a merge for that, just want you to notice.
@ -112,1 +166,4 @@
$('.js-generate').hidden = false;
$('.js-sign').hidden = false;
$('textarea[name="jwk"]').value = JSON.stringify({
Propietario

What's this hard-coded value for?

What's this hard-coded value for?
lastlink cerró este pull request 2019-05-11 22:32:00 +00:00
Inicie sesión para unirse a esta conversación.
No hay revisores
Sin etiquetas
Sin Milestone
No asignados
2 participantes
Notificaciones
Fecha de vencimiento
La fecha de vencimiento es inválida o está fuera de rango. Por favor utilice el formato 'aaaa-mm-dd'.

Sin fecha de vencimiento.

Dependencias

No se han establecido dependencias.

Referencia: coolaj86/bluecrypt-keypairs.js#10
No se ha proporcionado una descripción.