Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add EdDSA to JwsHeaderParams['alg'] type #194

Merged
merged 2 commits into from
Sep 21, 2023
Merged

add EdDSA to JwsHeaderParams['alg'] type #194

merged 2 commits into from
Sep 21, 2023

Conversation

mistermoe
Copy link
Member

This PR adds EdDSA (mentioned in RFC 8037) as an acceptable value for JwsHeaderParams['alg'].

image

@codecov
Copy link

codecov bot commented Aug 25, 2023

Codecov Report

Merging #194 (3bd18e9) into main (d56ec18) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #194   +/-   ##
=======================================
  Coverage   90.80%   90.81%           
=======================================
  Files          67       67           
  Lines       12828    12838   +10     
  Branches     1268     1268           
=======================================
+ Hits        11649    11659   +10     
  Misses       1156     1156           
  Partials       23       23           
Components Coverage Δ
api 94.32% <ø> (ø)
common 95.00% <ø> (ø)
credentials 92.77% <ø> (ø)
crypto 100.00% <100.00%> (ø)
dids 92.16% <ø> (ø)
agent 87.80% <ø> (ø)
identity-agent 59.05% <ø> (ø)
proxy-agent 58.59% <ø> (ø)
user-agent 57.36% <ø> (ø)

@frankhinek
Copy link
Contributor

frankhinek commented Aug 25, 2023

@mistermoe This change is also included in PR #175 but if there's a pressing need, we can certainly merge as a standalone PR.

@mistermoe
Copy link
Member Author

Oh nice! Na no worries then. Will close this

mistermoe and others added 2 commits September 21, 2023 16:42
@frankhinek
Copy link
Contributor

The change was unintentionally removed from PR #175 so merging this to add EdDSA

Copy link
Contributor

@frankhinek frankhinek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@frankhinek frankhinek merged commit 59a3045 into main Sep 21, 2023
22 of 23 checks passed
@frankhinek frankhinek deleted the jose-jws-header branch September 21, 2023 21:53
andorsk pushed a commit to modernfoundry/web5-js that referenced this pull request Sep 28, 2023
* `[crypto/jose]` add `EdDSA` to `JwsHeaderParams['alg']`

---------

Co-authored-by: Frank Hinek <frankhinek@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants