Skip to content

Enable Key Lookup #41

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 14 commits into
base: master
Choose a base branch
from
Open

Enable Key Lookup #41

wants to merge 14 commits into from

Conversation

jkevlin2
Copy link

  1. Export Parser.
  2. Have Parser initialize its instance.
  3. Add setSigningKeyId(kid) to Jwt.
  4. Allow Verifier.verify to accept a Jwt in addition to a string token.
  5. Tests for key-lookup, Parser and Verifier.
  6. Edited Readme, to show how a key lookup could be implemented.

@jkevlin2
export Parser and test
399fee9
@jkevlin2
Merge pull request #1 from jkevlin/issues/39
bd1761d 
export Parser and test
@jkevlin2
allow Verifier.verify to accept a Jwt as a parameter to bypass parsing
a73fc3a
@jkevlin2
Merge pull request #2 from jkevlin/verfy-accept-jwt-param
c6ea653 
allow Verifier.verify to accept a Jwt as a parameter to bypass parsing
@jkevlin2
add setter for keyid and test
efbaab3
@jkevlin2
Merge pull request #3 from jkevlin/add-setter-for-keyid
6ca2671 
add setter for keyid and test
@jkevlin2
added test and example in README for using a keyid for a lookup
a53b1d6
@jkevlin2
Merge pull request #4 from jkevlin/add-test-and-docs-for-keylookup
07fc6cc 
added test and example in README for using a keyid for a lookup
@jkevlin2
Per RFC changed from keyid to kid
cdeaa68
@jkevlin2
Merge pull request #5 from jkevlin/add-test-and-docs-for-keylookup
3f55b26 
Per RFC changed from keyid to kid
@coveralls
Copy link

coveralls commented Dec 25, 2016 •
edited
Loading

Coverage Status

Coverage remained the same at 100.0% when pulling 3f55b26 on jkevlin:master into e5ade5a on jwtk:master.

@coveralls
Copy link

coveralls commented Dec 25, 2016 •
edited
Loading

Coverage Status

Coverage remained the same at 100.0% when pulling 118a149 on jkevlin:master into e5ade5a on jwtk:master.

@coveralls
Copy link

coveralls commented Dec 25, 2016 •
edited
Loading

Coverage Status

Coverage remained the same at 100.0% when pulling 07e14e7 on jkevlin:master into e5ade5a on jwtk:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 07e14e7 on jkevlin:master into e5ade5a on jwtk:master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jkevlin2 @coveralls