Skip to content

Commit

Permalink
Merge pull request #648 from librespeed/633-invalid-username-password…
Browse files Browse the repository at this point in the history
…-variables-with-mssql-not-using-windows-authentication

fix: mysqlsetting were used for mssql
  • Loading branch information
sstidl authored Aug 10, 2024
2 parents 420be5e + d693074 commit 7327d2f
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion doc.md
Original file line number Diff line number Diff line change
Expand Up @@ -77,7 +77,7 @@ To use this function, you will need a database. The test supports MySQL, Postgre

This step is only required for MySQL, PostgreSQL and MSSQL. If you want to use SQLite, skip to the next step.

Log into your database using phpMyAdmin or a similar software and create a new database. Inside the `results` folder you will find `telemetry_mysql.sql`, `telemetry_postgresql.sql` and `telemetry_mssql.sql`, which are templates for MySQL and PostgreSQL respectively. Import the one you need, and you will see a `speedtest_users` table in the database. You can delete the templates afterwards.
Log into your database using phpMyAdmin or a similar software and create a new database. Inside the `results` folder you will find `telemetry_mysql.sql`, `telemetry_postgresql.sql` and `telemetry_mssql.sql`, which are templates for MySQL, PostgreSQL and MSSQL respectively. Import the one you need, and you will see a `speedtest_users` table in the database. You can delete the templates afterwards.

##### Configuring telemetry

Expand Down
2 changes: 1 addition & 1 deletion results/telemetry_db.php
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,7 @@ function getPdo($returnErrorMessage = false)
if($MsSql_WindowsAuthentication){
return new PDO($dsn, "", "", $pdoOptions);
} else {
return new PDO($dsn, $MySql_username, $MySql_password, $pdoOptions);
return new PDO($dsn, $MsSql_username, $MsSql_password, $pdoOptions);
}
}

Expand Down

0 comments on commit 7327d2f

Please sign in to comment.