LTI 1.3

doc/dev_doc/extensions_doc/plugins.rst

@@ -27,7 +27,7 @@ The following code adds a new page displaying ``This is a simple demo plugin`` o
 
     def init(plugin_manager, taskset_factory, client, plugin_config):
         """ Init the plugin """
-        plugin_manager.add_page("/<cookieless:sessionid>plugindemo", DemoPage.as_view('demopage'))
+        plugin_manager.add_page("/plugindemo", DemoPage.as_view('demopage'))
 
 
 The plugin is initialized by the plugin manager, which is the frontend-extended hook manager, by calling method ``init``.

inginious/frontend/app.py

@@ -33,7 +33,7 @@
 from inginious.frontend.taskset_factory import create_factories
 from inginious.common.entrypoints import filesystem_from_config_dict
 from inginious.common.filesystems.local import LocalFSProvider
-from inginious.frontend.lti_outcome_manager import LTIOutcomeManager
+from inginious.frontend.lti_grade_manager import LTIGradeManager
 from inginious.frontend.task_problems import get_default_displayable_problem_types
 from inginious.frontend.task_dispensers.toc import TableOfContents
 from inginious.frontend.task_dispensers.combinatory_test import CombinatoryTest
@@ -100,20 +100,24 @@ def _put_configuration_defaults(config):
 
     return config
 
+def get_homepath():
+    """ Returns the URL root. """
+    return flask.request.url_root[:-1]
 
-def get_homepath(ignore_session=False, force_cookieless=False):
+def get_path(*path_parts, force_cookieless=False):
     """
-    :param ignore_session: Ignore the cookieless session_id that should be put in the URL
+    :param path_parts: List of elements in the path to be separated by slashes
     :param force_cookieless: Force the cookieless session; the link will include the session_creator if needed.
     """
     session = flask.session
     request = flask.request
-    if not ignore_session and session.sid is not None and session.cookieless:
-        return request.url_root[:-1] + "/@" + session.sid + "@"
-    elif not ignore_session and force_cookieless:
-        return request.url_root[:-1] + "/@@"
-    else:
-        return request.url_root[:-1]
+    query_delimiter = '&' if path_parts and '?' in path_parts[-1] else '?'
+    path_parts = (get_homepath(), ) + path_parts
+    if session.sid is not None and session.cookieless:
+        return "/".join(path_parts) + f"{query_delimiter}session_id={session.sid}"
+    if force_cookieless:
+        return "/".join(path_parts) + f"{query_delimiter}session_id="
+    return "/".join(path_parts)
 
 
 def _close_app(mongo_client, client):
@@ -188,6 +192,7 @@ def get_app(config):
     if config.get("maintenance", False):
         template_helper = TemplateHelper(PluginManager(), None, config.get('use_minified_js', True))
         template_helper.add_to_template_globals("get_homepath", get_homepath)
+        template_helper.add_to_template_globals("get_path", get_path)
         template_helper.add_to_template_globals("pkg_version", __version__)
         template_helper.add_to_template_globals("available_languages", available_languages)
         template_helper.add_to_template_globals("_", _)
@@ -227,9 +232,9 @@ def get_app(config):
 
     client = create_arch(config, fs_provider, zmq_context, taskset_factory)
 
-    lti_outcome_manager = LTIOutcomeManager(database, user_manager, course_factory)
+    lti_grade_manager = LTIGradeManager(database, flask_app)
 
-    submission_manager = WebAppSubmissionManager(client, user_manager, database, gridfs, plugin_manager, lti_outcome_manager)
+    submission_manager = WebAppSubmissionManager(client, user_manager, database, gridfs, plugin_manager, lti_grade_manager)
     template_helper = TemplateHelper(plugin_manager, user_manager, config.get('use_minified_js', True))
 
     register_utils(database, user_manager, template_helper)
@@ -244,6 +249,7 @@ def get_app(config):
     template_helper.add_to_template_globals("str", str)
     template_helper.add_to_template_globals("available_languages", available_languages)
     template_helper.add_to_template_globals("get_homepath", get_homepath)
+    template_helper.add_to_template_globals("get_path", get_path)
     template_helper.add_to_template_globals("pkg_version", __version__)
     template_helper.add_to_template_globals("allow_registration", config.get("allow_registration", True))
     template_helper.add_to_template_globals("sentry_io_url", config.get("sentry_io_url"))
@@ -283,7 +289,7 @@ def flask_internalerror(e):
     flask_app.register_error_handler(InternalServerError, flask_internalerror)
 
     # Insert the needed singletons into the application, to allow pages to call them
-    flask_app.get_homepath = get_homepath
+    flask_app.get_path = get_path
     flask_app.plugin_manager = plugin_manager
     flask_app.taskset_factory = taskset_factory
     flask_app.course_factory = course_factory
@@ -299,7 +305,7 @@ def flask_internalerror(e):
     flask_app.default_max_file_size = default_max_file_size
     flask_app.backup_dir = config.get("backup_directory", './backup')
     flask_app.webterm_link = config.get("webterm", None)
-    flask_app.lti_outcome_manager = lti_outcome_manager
+    flask_app.lti_grade_manager = lti_grade_manager
     flask_app.allow_registration = config.get("allow_registration", True)
     flask_app.allow_deletion = config.get("allow_deletion", True)
     flask_app.available_languages = available_languages

inginious/frontend/courses.py

@@ -7,8 +7,9 @@
 
 import copy
 import gettext
+import hashlib
 import re
-from typing import List
+from typing import Iterable, List
 
 from inginious.frontend.user_settings.course_user_setting import CourseUserSetting
 from inginious.common.tags import Tag
@@ -18,6 +19,7 @@
 from inginious.frontend.task_dispensers.toc import TableOfContents
 from inginious.frontend.tasksets import _migrate_from_v_0_6
 
+from pylti1p3.tool_config import ToolConfDict
 
 class Course(object):
     """ A course with some modification for users """
@@ -54,7 +56,7 @@ def __init__(self, courseid, content, taskset_factory, task_factory, plugin_mana
         self._allow_preview = self._content.get('allow_preview', False)
         self._is_lti = self._content.get('is_lti', False)
         self._lti_url = self._content.get('lti_url', '')
-        self._lti_keys = self._content.get('lti_keys', {})
+        self._lti_config = self._content.get('lti_config', {})
         self._lti_send_back_grade = self._content.get('lti_send_back_grade', False)
         self._tags = {key: Tag(key, tag_dict, self.gettext) for key, tag_dict in self._content.get("tags", {}).items()}
         self._course_user_setting = {key: CourseUserSetting(key,
@@ -82,7 +84,7 @@ def __init__(self, courseid, content, taskset_factory, task_factory, plugin_mana
             self._groups_student_choice = False
             self._allow_unregister = False
         else:
-            self._lti_keys = {}
+            self._lti_config = {}
             self._lti_url = ''
             self._lti_send_back_grade = False
 
@@ -162,9 +164,28 @@ def is_lti(self):
         """ True if the current course is in LTI mode """
         return self._is_lti
 
-    def lti_keys(self):
-        """ {name: key} for the LTI customers """
-        return self._lti_keys if self._is_lti else {}
+    def lti_config(self):
+        """ LTI Tool config dictionary. Specs are at https://github.com/dmitry-viskov/pylti1.3/blob/master/README.rst?plain=1#L70-L98 """
+        return self._lti_config if self._is_lti else {}
+
+    def lti_tool(self) -> ToolConfDict:
+        """ LTI Tool object. """
+        lti_tool = ToolConfDict(self._lti_config)
+        for iss in self._lti_config:
+            for client_config in self._lti_config[iss]:
+                lti_tool.set_private_key(iss, client_config['private_key'], client_id=client_config['client_id'])
+                lti_tool.set_public_key(iss, client_config['public_key'], client_id=client_config['client_id'])
+        return lti_tool
+
+    def lti_platform_instances_ids(self) -> Iterable[str]:
+        """ Set of LTI Platform instance ids registered for this course. """
+        for iss in self._lti_config:
+            for client_config in self._lti_config[iss]:
+                for deployment_id in client_config['deployment_ids']:
+                    yield '/'.join([iss, client_config['client_id'], deployment_id])
+
+    def lti_keyset_hash(self, issuer: str, client_id: str) -> str:
+        return hashlib.md5((issuer + client_id).encode('utf-8')).digest().hex()
 
     def lti_url(self):
         """ Returns the URL to the external platform the course is hosted on """