You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Following a discussion in Slack (see PDF here or in Google Drive), the PR for #1387 was reverted because Canvas requires JWKS support even when it's not going to be used.
The intent of #1387 was to remove definition of LTIException, referrals to it, apparently unused JWKS functions, and cleaning up (removing) the LTI scope. It turns out that Canvas requires the JWKS support functions, even though the PR had been approved.
In this issue, investigate and determine whether these things can be done.
– Remove JWKS support. Does Canvas really require this, even when it's not used by the application? If the support cannot be removed, can it be reduced to something simple, like returning a dummy value?
– Remove LTIException definition. Is this exception definition really necessary? Can it be removed or left uncaught?
Originally posted by @lsloan in #1421 (comment)
The intent of #1387 was to remove definition of
LTIException
, referrals to it, apparently unused JWKS functions, and cleaning up (removing) the LTI scope. It turns out that Canvas requires the JWKS support functions, even though the PR had been approved.In this issue, investigate and determine whether these things can be done.
LTIException
definition. Is this exception definition really necessary? Can it be removed or left uncaught?The text was updated successfully, but these errors were encountered: