Uploaded image for project: 'Moodle app'
  1. Moodle app
  2. MOBILE-3458

Validate URL for custom URLs and QR reader when app has fixed sites

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 3.9.0
    • Fix Version/s: 3.9.1
    • Component/s: Custom URL Scheme
    • Labels:
    • Testing Instructions:
      Hide

      This issue only needs to be tested in 1 device (Android or iOS).

      Test default app

      1. In the app without any fixed site, go to add a site and click to scan a QR code.
      2. Scan a QR code pointing to any Moodle site (the URL must use the app's custom URL scheme, like it's done in user profile in Moodle 3.9.0).
      3. Check that the app let's you add the site.

      Test "branded" app with fixed sites

      1. Generate an app with at least 1 fixed site.
      2. Go to add a site and click to scan a QR code.
      3. Scan a QR code pointing to a Moodle site that's in the list of sites for the branded (the URL must use the app's custom URL scheme, like it's done in user profile in Moodle 3.9.0). Check that the app lets you add the site.
      4. Now repeat the test but using a QR code pointing to a Moodle site that isn't in the list of sites for the branded app, but using the same URL scheme. E.g. copy the URL of the QR code from master site and change the URL scheme to the one of your app.
      5. Check that the app displays an error.

      Test branded app with site finder

      1. Generate a "branded" app with the site finder and setting onlyallowlistedsites to true in config.json.
      2. Go to add a site and click to scan a QR code.
      3. Scan a QR code pointing to a Moodle site that's in the list of the site finder (the URL must use the app's custom URL scheme, like it's done in user profile in Moodle 3.9.0). Check that the app lets you add the site.
      4. Now repeat the test but using a QR code pointing to a Moodle site that isn't in the site finder, but using the app's custom URL scheme. Check that the app displays an error.
      Show
      This issue only needs to be tested in 1 device (Android or iOS). Test default app In the app without any fixed site, go to add a site and click to scan a QR code. Scan a QR code pointing to any Moodle site (the URL must use the app's custom URL scheme, like it's done in user profile in Moodle 3.9.0). Check that the app let's you add the site. Test "branded" app with fixed sites Generate an app with at least 1 fixed site. Go to add a site and click to scan a QR code. Scan a QR code pointing to a Moodle site that's in the list of sites for the branded (the URL must use the app's custom URL scheme, like it's done in user profile in Moodle 3.9.0). Check that the app lets you add the site. Now repeat the test but using a QR code pointing to a Moodle site that isn't in the list of sites for the branded app, but using the same URL scheme. E.g. copy the URL of the QR code from master site and change the URL scheme to the one of your app. Check that the app displays an error. Test branded app with site finder Generate a "branded" app with the site finder and setting onlyallowlistedsites to true in config.json. Go to add a site and click to scan a QR code. Scan a QR code pointing to a Moodle site that's in the list of the site finder (the URL must use the app's custom URL scheme, like it's done in user profile in Moodle 3.9.0). Check that the app lets you add the site. Now repeat the test but using a QR code pointing to a Moodle site that isn't in the site finder, but using the app's custom URL scheme. Check that the app displays an error.
    • Affected Branches:
      MOODLE_39_STABLE
    • Fixed Branches:
      MOODLE_39_STABLE
    • Pull Master Branch:
    • Sprint:
      Moodle App 3.9.1

      Description

      If an app has one or several fixed sites (e.g. branded app) we should validate that the custom URLs and the QR codes read belong to the sites of the app.

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              dpalou Dani Palou
              Reporter:
              dpalou Dani Palou
              Peer reviewer:
              Pau Ferrer
              Integrator:
              Juan Leyva
              Tester:
              Juan Leyva
              Participants:
              Component watchers:
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Fix Release Date:
                3/Jul/20