Modifies Materia OboNode server-side code to work with the upcoming Materia release.#2139
Draft
FrenjaminBanklin wants to merge 7 commits intomasterfrom
Draft
Modifies Materia OboNode server-side code to work with the upcoming Materia release.#2139FrenjaminBanklin wants to merge 7 commits intomasterfrom
FrenjaminBanklin wants to merge 7 commits intomasterfrom
Conversation
…ent it's actually necessary.
Modifies Materia Obonode to support LTI 1.3 version.
…oming Canvas header requirements.
…ly continue supporting the older version of Materia. Restored functionality necessary to support Materia via LTI1.1.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Materia's upcoming release removes support for LTI 1.1 in favor of LTI 1.3, requiring changes on the Obojobo end to have it act as a basic LTI 1.3 platform.
This update also pins a more recent commit of the upstream
ims-ltirepository to add newly required headers to outcomes requests sent to an LMS upon completing a module assessment.This requires a few additional environment variables:
MATERIA_LTI_VERSION: Either1.1or1.3.OBO_LTI_CLIENTID: This should be an alphanumeric string value determined by the platform (Obojobo) that needs to be matched in a corresponding LTI registration in any relevant tools. It can be an arbitrary alphanumeric string.OBO_LTI_UUID: This is a UUID value that is generated by the Materia LTI registration.OBO_JWT_KEY: This is an alphanumeric string value.OBO_PRIVATE_RSA_KEY: This is an RSA key generated by a command likeopenssl genrsa -out private.pem 2048. Ideally condensed to a single string with newline characters and bookends preserved.OBO_LTI_DEPLOYMENTID: This is a UUID value that is generated by the Materia LTI registration.