Skip to content
This repository has been archived by the owner on Feb 25, 2019. It is now read-only.

JWK Updates #2

Merged
merged 10 commits into from
Aug 19, 2017
Merged

JWK Updates #2

merged 10 commits into from
Aug 19, 2017

Conversation

EternalDeiwos
Copy link
Member

  • Made JWKSet.find and JWKSet.filter synchronous.
  • Use sift queries for find and filter object predicates.
  • JWKSet.publicJwks getter for publishing /jwks endpoint.

@codecov
Copy link

codecov bot commented Aug 19, 2017

Codecov Report

Merging #2 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@          Coverage Diff          @@
##           master     #2   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files           6      6           
  Lines         154    156    +2     
  Branches       26     25    -1     
=====================================
+ Hits          154    156    +2
Impacted Files Coverage Δ
src/JWK.js 100% <100%> (ø) ⬆️
src/JWKSet.js 100% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d1eaf77...4e8bfd7. Read the comment docs.

@EternalDeiwos
Copy link
Member Author

  • use nock for url importKey testing
  • generate kid's for keys generated using JWKSet

@christiansmith christiansmith merged commit 60ff94f into master Aug 19, 2017
@christiansmith christiansmith deleted the linklater-jwk-updates branch August 19, 2017 14:17
@christiansmith
Copy link
Member

Excellent, thanks!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants