no-acme #10

lastlink · 2019-05-11T20:15:44Z

lastlink okomentoval

2019-05-11 20:15:44 +00:00

Spolupracovník

fixed merge issue
jwk to jwt & jws

* fixed merge issue * jwk to jwt & jws

coolaj86 schválil tyto změny 2019-05-11 21:58:39 +00:00

coolaj86 zanechal komentář

Vlastník

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

app.js

						
				@ -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

coolaj86 okomentoval

2019-05-11 21:55:10 +00:00

Vlastník

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

app.js

						
				@ -112,1 +166,4 @@

				    $('.js-generate').hidden = false;

				    $('.js-sign').hidden = false;

				    $('textarea[name="jwk"]').value = JSON.stringify({

coolaj86 okomentoval

2019-05-11 21:54:43 +00:00

Vlastník

What's this hard-coded value for?

lastlink uzavřel/a tento pull request

2019-05-11 22:32:00 +00:00

Přihlaste se pro zapojení do konverzace.

2 účastníků

Oznámení

Termín dokončení

Žádný termín dokončení.

Závislosti

Nejsou nastaveny žádné závislosti.

Reference: coolaj86/bluecrypt-keypairs.js#10