1
1
Fork 1

no-acme #10

Merge aplicado
lastlink aplicou merge dos 13 commits de lastlink/bluecrypt-keypairs.js:no-acme em master 2019-05-11 22:32:00 +00:00
Collaborator
  • fixed merge issue
  • jwk to jwt & jws
* fixed merge issue * jwk to jwt & jws
coolaj86 aprovou estas alterações 2019-05-11 21:58:39 +00:00
coolaj86 deixou um comentário
Proprietário

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
Proprietário

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({
Proprietário

What's this hard-coded value for?

What's this hard-coded value for?
lastlink fechou este pull request 2019-05-11 22:32:00 +00:00
Acesse para participar desta conversação.
Sem revisor
Sem etiqueta
Sem marco
Sem responsável
2 participante(s)
Notificações
Data limite
A data limite é inválida ou está fora do intervalo. Por favor, use o formato 'dd/mm/aaaa'.

Data limite não informada.

Dependências

Nenhuma dependência definida.

Referência: coolaj86/bluecrypt-keypairs.js#10
Nenhuma descrição fornecida.