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 DPoP createJWT and buildDPoPHeaders function #187

Merged
merged 2 commits into from
Feb 22, 2024
Merged

add DPoP createJWT and buildDPoPHeaders function #187

merged 2 commits into from
Feb 22, 2024

Conversation

jshawl
Copy link
Contributor

@jshawl jshawl commented Feb 15, 2024

This PR is a follow-up PR to #186 and is the last of the DPoP related changes in sdk-client.

This PR implements the JWT creation and signature and exports a single method to consumers of this package, buildDPoPHeaders, that abstracts away all of the lower-level cryptographic and encoding related functionality.

Copy link

codecov bot commented Feb 15, 2024

Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (c97c16f) 72.40% compared to head (4cd75a5) 73.66%.

Files Patch % Lines
src/index.js 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #187      +/-   ##
==========================================
+ Coverage   72.40%   73.66%   +1.25%     
==========================================
  Files          23       23              
  Lines        1939     2035      +96     
  Branches      236      240       +4     
==========================================
+ Hits         1404     1499      +95     
- Misses        525      526       +1     
  Partials       10       10              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@jshawl jshawl changed the title add createJWT function add DPoP createJWT function Feb 15, 2024
@jshawl jshawl changed the title add DPoP createJWT function add DPoP createJWT and buildDPoPHeaders function Feb 16, 2024
@jshawl jshawl marked this pull request as ready for review February 21, 2024 14:48
@jshawl jshawl requested a review from a team as a code owner February 21, 2024 14:48
Base automatically changed from dpop-keygen to main February 22, 2024 18:12
alphabetize import and clarify base64 test case

add createJWT function

add buildDPoPHeaders function and export it
@jshawl jshawl merged commit 639b4d5 into main Feb 22, 2024
5 checks passed
@jshawl jshawl deleted the dpop-jwt branch February 22, 2024 19:03
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.

2 participants