Skip to content

Commit

Permalink
Remove JWK "alg" field (#24)
Browse files Browse the repository at this point in the history
The `alg` field is not required and might change in the future, see
https://www.ietf.org/archive/id/draft-jones-jose-fully-specified-algorithms-00.html.

* Revert "Set and check "alg" JWK property for EdDSA keys (#3)"

This reverts commit bf5f161.

* Revert "Update JWK mappings for Ed25519 and Ed448 (#4)"

This reverts commit 5d58d49.
  • Loading branch information
twiss committed Aug 30, 2023
1 parent 36c0f23 commit c9eaaac
Showing 1 changed file with 2 additions and 32 deletions.
34 changes: 2 additions & 32 deletions index.html
Original file line number Diff line number Diff line change
Expand Up @@ -2241,14 +2241,6 @@ <h4>Operations</h4>
{{DataError}}.
</p>
</li>
<li>
<p>
If the {{JsonWebKey/alg}} field of |jwk| is present and is
not "`EdDSA`",
then [= exception/throw =] a
{{DataError}}.
</p>
</li>
<li>
<p>
If |usages| is non-empty and the {{JsonWebKey/use}} field of |jwk| is present and is
Expand Down Expand Up @@ -2543,12 +2535,6 @@ <h4>Operations</h4>
"`OKP`".
</p>
</li>
<li>
<p>
Set the `alg` attribute of |jwk| to
"`EdDSA`".
</p>
</li>
<li>
<p>
Set the `crv` attribute of |jwk| to
Expand Down Expand Up @@ -3148,14 +3134,6 @@ <h4>Operations</h4>
{{DataError}}.
</p>
</li>
<li>
<p>
If the {{JsonWebKey/alg}} field of |jwk| is present and is
not "`EdDSA`",
then [= exception/throw =] a
{{DataError}}.
</p>
</li>
<li>
<p>
If |usages| is non-empty and the {{JsonWebKey/use}} field of |jwk| is present and is
Expand Down Expand Up @@ -3450,12 +3428,6 @@ <h4>Operations</h4>
"`OKP`".
</p>
</li>
<li>
<p>
Set the `alg` attribute of |jwk| to
"`EdDSA`".
</p>
</li>
<li>
<p>
Set the `crv` attribute of |jwk| to
Expand Down Expand Up @@ -3638,8 +3610,7 @@ <h3>Algorithm mappings</h3>
<td>
<pre class=js>
{ kty: "OKP",
crv: "Ed25519",
alg: "EdDSA" }
crv: "Ed25519" }
</pre>
</td>
<td>
Expand All @@ -3652,8 +3623,7 @@ <h3>Algorithm mappings</h3>
<td>
<pre class=js>
{ kty: "OKP",
crv: "Ed448",
alg: "EdDSA" }
crv: "Ed448" }
</pre>
</td>
<td>
Expand Down

0 comments on commit c9eaaac

Please sign in to comment.