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

              • Votes:
                4 Vote for this issue
                Watchers:
                6 Start watching this issue

                Dates

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