no-acme #10

Samengevoegd
lastlink heeft 13 commits samengevoegd van lastlink/bluecrypt-keypairs.js:no-acme naar 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 heeft deze veranderingen 2019-05-11 21:58:39 +00:00 goedgekeurd
coolaj86 heeft een reactie achtergelaten
Eigenaar

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
Eigenaar

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({
Eigenaar

What's this hard-coded value for?

What's this hard-coded value for?
lastlink heeft deze pull request gesloten 2019-05-11 22:32:00 +00:00
Log in om deel te nemen aan deze discussie.
Geen beoordelaars
Geen label
Geen mijlpaal
Niet toegewezen
2 deelnemers
Notificaties
Vervaldatum
De deadline is ongeldig of buiten bereik. Gebruik het formaat 'jjjj-mm-dd'.

Geen vervaldatum ingesteld.

Afhankelijkheden

Geen afhankelijkheden ingesteld.

Referentie: coolaj86/bluecrypt-keypairs.js#10
No description provided.