Uploaded image for project: 'Sakai'
  1. Sakai
  2. SAK-43267

LTI 1.3 - If the OIDC endpoint contains parameters the redirect URL is not built correctly.

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: RESOLVED
    • Priority: Blocker
    • Resolution: Fixed
    • Affects Version/s: 12.7, 19.3, 20.0, 21.0 [Tentative]
    • Fix Version/s: 20.0, 21.0 [Tentative]
    • Component/s: BasicLTI
    • Labels:
      None
    • 20 status:
      Resolved
    • Test Plan:
      Hide

      When configuring an LTI 1.3 tool, configure the OIDC login initiation url to have a query parameter. For example:

      https://52.13.0.113:8443/lti13-test/oidclaunch?lti_deployment_id=testdeploy

      Once configured, try to launch the tool (including if the tool doesn't exist it should try to launch it)

      Then, Sakai generates the definitive url adding the iss and the login hint, but the current code adds a ? instead a & before the iss.

      Check that the url called has the right format using only one ? in front the first parameters, and keeps the lti_deployment_id=testdeploy parameter correctly.

      Show
      When configuring an LTI 1.3 tool, configure the OIDC login initiation url to have a query parameter. For example: https://52.13.0.113:8443/lti13-test/oidclaunch?lti_deployment_id=testdeploy Once configured, try to launch the tool (including if the tool doesn't exist it should try to launch it) Then, Sakai generates the definitive url adding the iss and the login hint, but the current code adds a ? instead a & before the iss. Check that the url called has the right format using only one ? in front the first parameters, and keeps the lti_deployment_id=testdeploy parameter correctly.

      Gliffy Diagrams

        Zeplin

          Attachments

            Activity

              People

              • Assignee:
                farreri Miguel Pellicer
                Reporter:
                ddelblanco Diego del Blanco Orobitg
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Git Integration