-
Notifications
You must be signed in to change notification settings - Fork 105
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
Switch from JWT to Server-to-Server OAuth before June 2023 #387
Comments
Beyond the technical requirements, a communication plan would be valuable for Moodle administrators. The Zoom administrator may be in an entirely separate part of the organization and will need to get the new app type configured. The release that includes this functionality should:
|
Just going to leave https://docs.moodle.org/400/en/OAuth_2_services here as it looks like it may be feasible to use Moodle's Oauth 2 services to define how to connect to Zoom |
Testing completed, no issues found. Verified the basic functionality and the scheduled tasks. |
What does configuring this look like on the Zoom end? I started created a server-to-server OAuth app but when I got to the step where you add scopes I wasn't sure what to do. What scopes does this need? Is there plan to write up a wiki page or something on what it looks like to switch to OAuth? No rush obviously I just want to be prepared. |
@haietza oops, my apologies. I read the issue, looked at the wiki, but didn't think to check if the readme was updated. Thank you! |
My summary of https://marketplace.zoom.us/docs/guides/build/jwt-app/jwt-faq/
Overall, we'll want to have a version that uses Server-to-Server OAuth by March 2023 or sooner so people have a chance to install and configure the updated integration. It would also be good to:
We'll likely need a new field for each element required to have a fresh token and potentially leverage an OAuth library as needed. If an OAuth library is needed and Moodle already provides one, this may require an increase in our minimum supported Moodle version.
If anyone else has thoughts on this, please share.
The text was updated successfully, but these errors were encountered: