no-acme #10

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

lastlink прокомментировал(а)

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

Соавтор

fixed merge issue
jwk to jwt & jws

* fixed merge issue * jwk to jwt & jws

coolaj86 одобрил(а) эти изменения 2019-05-11 21:58:39 +00:00

coolaj86 оставил комментарий

Владелец

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 прокомментировал(а)

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

Владелец

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 прокомментировал(а)

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

Владелец

What's this hard-coded value for?

lastlink закрыл этот запрос на слияние

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

Войдите, чтобы присоединиться к обсуждению.

2 участников

Уведомления

Срок выполнения

Срок выполнения не установлен.

Зависимости

Зависимостей нет.

Ссылка: coolaj86/bluecrypt-keypairs.js#10