no-acme #10

マージ済み
lastlink が 13 個のコミットを lastlink/bluecrypt-keypairs.js:no-acme から master へマージ 2019-05-11 22:32:00 +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?

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
オーナー

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({
オーナー

What's this hard-coded value for?

What's this hard-coded value for?
lastlink がプルリクエストをクローズ 2019-05-11 22:32:00 +00:00
サインインしてこの会話に参加。
レビューアなし
ラベルなし
マイルストーンなし
担当者なし
2 人の参加者
通知
期日
期日が正しくないか範囲を超えています。 'yyyy-mm-dd' の形式で入力してください。

期日は未設定です。

依存関係

依存関係が設定されていません。

リファレンス: coolaj86/bluecrypt-keypairs.js#10
説明はありません。