fix: LTI Deep linking launch parameter for H5P

README.rst

@@ -367,6 +367,13 @@ Changelog
 
 Please See the [releases tab](https://github.com/edx/xblock-lti-consumer/releases) for the complete changelog.
 
+3.4.4 - 2022-03-03
+------------------
+
+* Fix LTI 1.3 Deep Linking launch url - always perform launch on launch URL, but update `target_link_uri` when
+  loading deep linking content.
+  See LTI 1.3 spec at: https://www.imsglobal.org/spec/lti/v1p3#target-link-uri
+
 3.4.3 - 2022-02-01
 ------------------
 

lti_consumer/__init__.py

@@ -4,4 +4,4 @@ Runtime will load the XBlock class from here.
 from .apps import LTIConsumerApp
 from .lti_xblock import LtiConsumerXBlock
 
-__version__ = '3.4.3'
+__version__ = '3.4.4'

lti_consumer/lti_xblock.py

@@ -1145,8 +1145,6 @@ class LtiConsumerXBlock(StudioEditableXBlockMixin, XBlock):
                     url=dl_params.get('url'),
                     custom=dl_params.get('custom')
                 )
-                if dl_params.get('url'):
-                    context.update({'launch_url': dl_params.get('url')})
 
             # Update context with LTI launch parameters
             context.update({

lti_consumer/tests/unit/test_lti_xblock.py

@@ -1416,7 +1416,7 @@ class TestLtiConsumer1p3XBlock(TestCase):
 
         # Check response
         self.assertEqual(response.status_code, 200)
-        self.assertIn("https://deep-link-content/", content)
+        self.assertIn("http://tool.example/launch", content)
 
 
 class TestLti1p3AccessTokenEndpoint(TestLtiConsumerXBlock):