-
Notifications
You must be signed in to change notification settings - Fork 65
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
Allowing JWE aswell as JWT tokens #66
Comments
Would that require a new dependency? It doesn't look like |
Yes it would. We currently use https://github.com/cisco/node-jose |
@uplift If I am reading this correctly, we would be removing our dependency on Seems like a fairly major operation, and wouldn't be backwards compatible with pre-existing installs, unless |
Hi @julianlam. I am curious if it's possible to add this feature as well. This library https://github.com/panva/jose seems potentially more widely used than the one mentioned above and can produce a JWS for backwards compatibility. It would also allow for different signing algorithms as requested in issue #31. I'm not an expert on JWT's or nodebb but I'd be willing to help if you're interested. |
In my quest to bring our custom version of this project inline with the official release, I have one more task to make them the same.
Our payload cookie is currently encoded using JWE tokens instead of basic JWT. Would it be possible to add a config in the Admin Setting page that allows the switch to JWE encryption?
The text was updated successfully, but these errors were encountered: