Releases: openedx/xblock-lti-consumer
9.0.0 Validation on launch redirect URIs
fix[olive]: backport LTI 1.3 grade injection vulnerability fix
What's Changed
- Merge pull request from GHSA-7j9p-67mm-5g87 by @mtyaka in #331
New Contributors
Full Changelog: v4.5.0...4.5.1
Fix inappropriate instructions for inline PII sharing consent dialog
This release fixes inappropriate instructions displayed when collecting PII sharing consent before an inline LTI launch. Previously, the instructions said, "Click OK to have your [username (and) e-mail address] sent to a 3rd party application. Click Cancel to return to this page without sending your information." The latter sentence does not make sense in the context of an inline launch, because there is no cancel button. This commit modifies the inline PII sharing consent dialog to say, "Click OK to have your [username (and) e-mail address] sent to a 3rd party application."
v8.0.0
refactor: fix module-containing imports
What's Changed
Full Changelog: 7.2.3...7.3.0
Fix bug in rending buttons and message in PII sharing consent dialog.
This release fixes a bug in the way that the PII sharing consent dialog renders. The bug resulted in the "OK" and "Cancel" buttons as well as the text of the PII sharing consent prompt appearing inside an inappropriate component when there was more than one LTI component in a unit.
7.2.2 Fixes LTI 1.3 grade injection vulnerability
7.2.1 URL configuration support
feat: new release for url settings (#323)
Fix PII Sharing Behavior and Enable PII Sharing in LTI 1.3 Launches and Fix LTI 1.3 Modal Launches
This release addresses a number of issues with and bugs in sharing personally identifiable information (PII) in LTI
launches.
-
Replaces the PII sharing consent modal with an inline PII sharing consent dialog to better suit the three different
LTI launch types (i.e.inline
,modal
, andnew_window
). -
Adds a PII consent dialog for
inline
LTI launches. -
Fixes a bug in the
modal
LTI launch in LTI 1.3 that was preventing the LTI launch. -
Fixes a bug in evaluating and caching whether PII sharing is enabled via the
CourseAllowPIISharingInLTIFlag
.- This fixes a bug where the PII sharing fields in the LTI XBlock edit menu appeared regardless of the existence or
value of this flag. The PII sharing fields will now always be hidden if either noCourseAllowPIISharingInLTIFlag
exists for a course or if aCourseAllowPIISharingInLTIFlag
exists for the course but is not enabled. - This fixes a bug in the backwards compatibility code in
lti_access_to_learners_editable
. Now,
CourseAllowPIISharingInLTIFlag
will always be created for courses that contain (an) LTI XBlock(s) that have (a)
PII sharing field(s) set to True when a user opens the LTI XBlock edit menu. Before, this would occur inconsistently
due to a bug in the caching code.
- This fixes a bug where the PII sharing fields in the LTI XBlock edit menu appeared regardless of the existence or
-
Enables sharing username and email in LTI 1.3 launches.
- Adds
preferred_username
andemail
attributes to theLti1p3LaunchData
class. The application or context
that instantiatesLti1p3LaunchData
is responsible for ensuring that username and email can be sent via an LTI
1.3 launch and supplying these data, if appropriate.
- Adds
-
Adds code to eventually support the value of
CourseAllowPIISharingInLTIFlag
controlling PII sharing for a given
course in LTI 1.1 and LTI 1.3 launches.- This code does not currently work, because the LTI configuration service is not available or defined in all runtime
contexts. This code works in the LTI XBlock edit menu (i.e. thestudio_view
), but it does not work in the Studio
preview context (i.e. theauthor_view
) or the LMS (i.e. thestudent_view
). The effect is that
theCourseAllowPIISharingInLTIFlag
can only control the appearance of the username and email PII sharing fields in
the XBlock edit menu; it does not control PII sharing. We plan to fix this bug in the future.
- This code does not currently work, because the LTI configuration service is not available or defined in all runtime
7.1.0 Allow disabling NRPS pii
feat: adds setting to prevent nrps pii (#315) We would like to enable PII in an LTI1.3 launch but turning that flag on would allow the tool to grab PII for the entire course roster via NRPS. We have not fully evaluated the privacy concerns if that is allowed. For the time being this platform setting can wholly disable PII over NRPS to avoid the issue