#1009 mod_storage_sql does not iterate through prosodyarchive when user is deleted
Reporter
nico
Owner
Zash
Created
Updated
Stars
★ (1)
Tags
Status-Fixed
Type-Defect
Priority-Medium
Milestone-0.10
nico
on
What steps will reproduce the problem?
1. enable mam with mod_storage_sql
2. enable client to store message on the server
3. chat with ppl
4. delete your account
5. query the db for your old messages
What is the expected output?
function driver:purge(username) should iterate through the prosodyarchive db to delete old mam /offline messages from the db storage.
What do you see instead?
prosodyarchive is not iterated through.
What version of the product are you using? On what operating system?
prosody 0.10.0.1 latest deb from prosody repo
Please provide any additional information below.
This is the part that should list the prosody archive https://hg.prosody.im/0.10/file/0.10.0/plugins/mod_storage_sql.lua#l416
Something like
local stmt,err = engine:delete("DELETE FROM \"prosodyarchive\" WHERE \"host\"=? AND \"user\"=?", host, username);
What steps will reproduce the problem? 1. enable mam with mod_storage_sql 2. enable client to store message on the server 3. chat with ppl 4. delete your account 5. query the db for your old messages What is the expected output? function driver:purge(username) should iterate through the prosodyarchive db to delete old mam /offline messages from the db storage. What do you see instead? prosodyarchive is not iterated through. What version of the product are you using? On what operating system? prosody 0.10.0.1 latest deb from prosody repo Please provide any additional information below. This is the part that should list the prosody archive https://hg.prosody.im/0.10/file/0.10.0/plugins/mod_storage_sql.lua#l416 Something like local stmt,err = engine:delete("DELETE FROM \"prosodyarchive\" WHERE \"host\"=? AND \"user\"=?", host, username);
Thanks for the report, fixed in https://hg.prosody.im/0.10/rev/34814a908557
Changes