-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Support ES256 #2453
Labels
good first issue
A good issue to tackle when being a novice to the project.
help wanted
We are looking for help on this one.
Comments
Contribs welcomed! |
aeneasr
added
good first issue
A good issue to tackle when being a novice to the project.
help wanted
We are looking for help on this one.
labels
Apr 18, 2021
Yeah also not sure what this is referring to any more. Maybe we can close |
actually i don't see the changes merged in PR 628 in master now. Was it intentionally dropped ? |
@aeneasr Should I create a PR with the same changes to the |
That PR is from 2017, thats 5 years ago - it probably won’t work ;) |
Oh okay, I missed the date on the PR 😅. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
good first issue
A good issue to tackle when being a novice to the project.
help wanted
We are looking for help on this one.
So fosite has ES256 support, I have seen some past PRs to support ES256 in Hydra (#628, #627), but looking at the code I do not see it really being supported in Hydra.
While RS256 is recommended by the spec, ES256 is close to that with its "Recommended+" recommendation. Supporting ES256 would thus allow one to migrate away from RS256 to ES256.
The text was updated successfully, but these errors were encountered: