Join GitHub today
GitHub is home to over 50 million developers working together to host and review code, manage projects, and build software together.
Sign upAdd more docs about JWK support #341
Conversation
docs for the following code - 922dac6#diff-35b558119ace51c78f5563ad9f4d2ec3R8 - 922dac6#diff-35b558119ace51c78f5563ad9f4d2ec3R49
sourcelevel-bot
bot
commented
Nov 26, 2019
Hello, @take! This is your first Pull Request that will be reviewed by SourceLevel, an automatic Code Review service. It will leave comments on this diff with potential issues and style violations found in the code as you push new commits. You can also see all the issues found on this Pull Request on its review page. Please check our documentation for more information. |
or by passing JWK as a simple Hash | ||
|
||
``` | ||
jwks = { keys: [{ ... }] } # keys needs to be Symbol |
This comment has been minimized.
This comment has been minimized.
martinemde
Mar 25, 2020
Contributor
I added #348 to accept string keys. If that gets merged, this will need an update.
This comment has been minimized.
This comment has been minimized.
excpt
Jul 7, 2020
Member
If it's okay for you I would merge this doc update. Then you can rebase your PR and add the new documentation information.
@bkeepers would help if you can look into this since there's relative PRs already |
take commentedNov 26, 2019
docs for the following code