diff options
| author | Kim Alvefur <zash@zash.se> | 2017-04-13 01:29:35 +0200 |
|---|---|---|
| committer | Kim Alvefur <zash@zash.se> | 2017-04-13 01:29:35 +0200 |
| commit | caf3175a5f2b457c97bfe6ba217a2fcc0661baf2 (patch) | |
| tree | 836f0e76f9472d563d77d6c4464fb3905853b840 | |
| parent | 8aaba129330f4d197c648464cc9312c95b727553 (diff) | |
| download | prosody-caf3175a5f2b457c97bfe6ba217a2fcc0661baf2.tar.gz prosody-caf3175a5f2b457c97bfe6ba217a2fcc0661baf2.zip | |
mod_storage_sql: Limit encoding check to current database
| -rw-r--r-- | plugins/mod_storage_sql.lua | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/plugins/mod_storage_sql.lua b/plugins/mod_storage_sql.lua index 9f2933a0..aab43d1f 100644 --- a/plugins/mod_storage_sql.lua +++ b/plugins/mod_storage_sql.lua @@ -485,11 +485,12 @@ local function upgrade_table(engine, params, apply_changes) -- luacheck: ignore SELECT "COLUMN_NAME","COLUMN_TYPE","TABLE_NAME" FROM "information_schema"."columns" WHERE "TABLE_NAME" LIKE 'prosody%%' + AND "TABLE_SCHEMA" = ? AND ( "CHARACTER_SET_NAME"!=? OR "COLLATION_NAME"!=?); ]]; -- FIXME Is it ok to ignore the return values from this? engine:transaction(function() - local result = assert(engine:execute(check_encoding_query, engine.charset, engine.charset.."_bin")); + local result = assert(engine:execute(check_encoding_query, params.database, engine.charset, engine.charset.."_bin")); local n_bad_columns = result:rowcount(); if n_bad_columns > 0 then changes = true; @@ -508,7 +509,8 @@ local function upgrade_table(engine, params, apply_changes) -- luacheck: ignore end end); success,err = engine:transaction(function() - return engine:execute(check_encoding_query, engine.charset, engine.charset.."_bin"); + return engine:execute(check_encoding_query, params.database, + engine.charset, engine.charset.."_bin"); end); if not success then module:log("error", "Failed to check/upgrade database encoding: %s", err or "unknown error"); |