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

SQL Server connection error when using a specific port to a different subnetwork

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 3.3, 3.3.1
    • Fix Version/s: 3.2.5, 3.3.2
    • Component/s: Libraries
    • Labels:
    • Database:
      Microsoft SQL
    • Testing Instructions:
      Hide
      1. Bring up the MSSQL Docker image on the standard port
      2. Configure config.php for mssql, without specifying a port at all
      3. Attempt to install Moodle
      4. Bring down the Docker image, etc.
      5. Bring up the MSSQL Docker image on a custom port
      6. Configure config.php to point to that custom port
      7. Attempt to install Moodle
      Show
      Bring up the MSSQL Docker image on the standard port Configure config.php for mssql, without specifying a port at all Attempt to install Moodle Bring down the Docker image, etc. Bring up the MSSQL Docker image on a custom port Configure config.php to point to that custom port Attempt to install Moodle
    • Workaround:
      Hide

      the patch below fix this issue

      Show
      the patch below fix this issue
    • Affected Branches:
      MOODLE_33_STABLE
    • Fixed Branches:
      MOODLE_32_STABLE, MOODLE_33_STABLE
    • Pull Master Branch:
      MDL-59509-master

      Description

      There is a SQL Server connection error when using a specific port to a sqlsrv instance in a different subnetwork.

      Actually the $this->dboptions['dbport'] is not used arround line 186 into the:

      lib/mdl/sqlsrv_native_moodle_database::connect() function

       

       

       

        Attachments

          Activity

            People

            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:
                Fix Release Date:
                11/Sep/17