aboutsummaryrefslogtreecommitdiffstats
path: root/core/rostermanager.lua
diff options
context:
space:
mode:
authorKim Alvefur <zash@zash.se>2017-12-29 20:49:19 +0100
committerKim Alvefur <zash@zash.se>2017-12-29 20:49:19 +0100
commit76d18a84640f7083f658776d66a7833f067e0377 (patch)
tree0d7b74089aa23a80e48a7db72579dd733a93b703 /core/rostermanager.lua
parentb8915c9db4fb03d17da7023b0e70b9872811785a (diff)
downloadprosody-76d18a84640f7083f658776d66a7833f067e0377.tar.gz
prosody-76d18a84640f7083f658776d66a7833f067e0377.zip
rostermanager: Explicitly remove old-style 'pending' table from rosters (or it stays forever due to map stores)
Diffstat (limited to 'core/rostermanager.lua')
-rw-r--r--core/rostermanager.lua8
1 files changed, 8 insertions, 0 deletions
diff --git a/core/rostermanager.lua b/core/rostermanager.lua
index f6a44028..65be0de0 100644
--- a/core/rostermanager.lua
+++ b/core/rostermanager.lua
@@ -112,7 +112,15 @@ local function load_roster(username, host)
local data, err = roster_store:get(username);
roster = data or {};
if user then user.roster = roster; end
+ local legacy_pending = roster.pending and type(roster.pending.subscription) ~= "string";
roster_metadata(roster, err);
+ if legacy_pending then
+ -- Due to map store use, we need to manually delete this entry
+ log("debug", "Removing legacy 'pending' entry");
+ if not save_roster(username, host, roster, "pending") then
+ log("warn", "Could not remove legacy 'pendig' entry");
+ end
+ end
if roster[jid] then
roster[jid] = nil;
log("debug", "Roster for %s had a self-contact, removing", jid);