-
Notifications
You must be signed in to change notification settings - Fork 3
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
Replace pyjwkest #186
Comments
It looks like https://github.com/openedx/xblock-lti-consumer and https://github.com/edx/token-utils include it as well. I'm talking with the owners of edx-token-utils about switching over, and making sure their public and private keys are ready for the change. For reference, these are the two config-related issues that need to be addressed for each deployer:
|
@iamsobanjaved can you review and comment if this is still a valid issue to keep alive? If so, should it be in the Arch- or Arbi-BOM board? |
I created this issue to track it across multiple repos. It is currently pending in 2 repos; I will add those in tasklist. We will be picking up on pending repos this quarter. |
Waiting for the owning team testing on this issue. Tracking testing in the issue https://2u-internal.atlassian.net/browse/COSMO-537. |
The
pyjwkest
package was abandoned upstream more than 3 years ago. Let's replace it with something more actively maintained. First, check to see if the latest version of PyJWT (already used here) supports what we need frompyjwkest
. If not, replace it with cryptojwt as it is from the same author and actively maintained. Also, this package has a similar interface to the one which is being replaced.Tasks
The text was updated successfully, but these errors were encountered: