Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-58727

tool_mobile_get_public_config should return always the launchurl

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 3.3
    • Fix Version/s: 3.3
    • Component/s: Administration
    • Labels:
    • Testing Instructions:
      Hide
      1. Configure your installation to not allow signup, check that the typeoflogin setting of tool_mobile is configured to "Log-in via the app"
      2. Execute the following curl command pointing to your local installation

        curl 'http://localhost/m/stable_master/lib/ajax/service.php' --data-binary '[{"index":0,"methodname":"tool_mobile_get_public_config","args":{}}]' | python -m "json.tool"

      3. And check that the "launchurl" field is returned including the correct url to your admin/tool/mobile/launch.php file
      Show
      Configure your installation to not allow signup, check that the typeoflogin setting of tool_mobile is configured to "Log-in via the app" Execute the following curl command pointing to your local installation curl 'http://localhost/m/stable_master/lib/ajax/service.php' --data-binary ' [{"index":0,"methodname":"tool_mobile_get_public_config","args":{}}] ' | python -m "json.tool" And check that the "launchurl" field is returned including the correct url to your admin/tool/mobile/launch.php file
    • Affected Branches:
      MOODLE_33_STABLE
    • Fixed Branches:
      MOODLE_33_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-58727-master

      Description

      Recently we added support to the new OAuth SSO methods (Google, Facebook, Microsoft) and identity providers, that will require launchurl to be returned always by tool_mobile_get_public_config

      It is safe to return always the launchurl, the main reason to not return it was to reduce data returned by Web Services when was not required.

      This is required only for 3.3 (3.2 is not supported yet)

        Attachments

          Activity

            People

            Assignee:
            jleyva Juan Leyva
            Reporter:
            jleyva Juan Leyva
            Peer reviewer:
            Dani Palou
            Integrator:
            David Monllaó
            Tester:
            Jun Pataleta
            Participants:
            Component watchers:
            Andrew Nicols, Jun Pataleta, Michael Hawkins, Shamim Rezaie, Simey Lameze
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:
              Fix Release Date:
              15/May/17