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

External DB auth: IN sql clause can get too big

    XMLWordPrintable

    Details

    • Testing Instructions:
      Hide
      • Enable external DB authentication plugin
      • Set up the plugin on an external site containing more than 70.000 users (use the data generator in the Admin > Development > Generate course - you may want to edit the generator php script to allow the creation of 70.000 users instead of the default)
      • Set "Removed ext user" to "suspended" and in the field mapping section, 'update local' to 'on every login'. These two options should force the sync_users.php script to go through remove/update users (to be verified though)

      Run the aut/db/cli/sync_users.php

      Show
      Enable external DB authentication plugin Set up the plugin on an external site containing more than 70.000 users (use the data generator in the Admin > Development > Generate course - you may want to edit the generator php script to allow the creation of 70.000 users instead of the default) Set "Removed ext user" to "suspended" and in the field mapping section, 'update local' to 'on every login'. These two options should force the sync_users.php script to go through remove/update users (to be verified though) Run the aut/db/cli/sync_users.php
    • Affected Branches:
      MOODLE_27_STABLE, MOODLE_30_STABLE, MOODLE_31_STABLE, MOODLE_32_STABLE
    • Fixed Branches:
      MOODLE_30_STABLE, MOODLE_31_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      wip-MDL-46682-master

      Description

      In auth/db/auth.php - sync_users()

      If your sync operation allows to remove/update users, and if the number of user of the external db is above 65535, then the sync crashes.

      Error: number of parameters must be between 0 and 65535

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              abgreeve Adrian Greeve
              Reporter:
              jerome Jérôme Mouneyrac
              Peer reviewer:
              Adrian Greeve
              Integrator:
              Eloy Lafuente (stronk7)
              Tester:
              Rajesh Taneja
              Participants:
              Component watchers:
              Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias
              Votes:
              4 Vote for this issue
              Watchers:
              6 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Fix Release Date:
                11/Jul/16