#867 MAM: Enable archiving after users first query
Reporter
Ge0rG
Owner
Zash
Created
Updated
Stars
★★ (3)
Tags
Priority-Low
Type-Enhancement
Status-Fixed
Milestone-0.10
Ge0rG
on
Once mod_mam is loaded, it will log data for all users on the server, even for high-traffic users who never actually use MAM.
Please copy the behavior of ejabberd where MAM is only enabled for a given account once that user accesses the MAM archive once.
Zash
on
Changes
owner Zash
tags Status-Accepted Milestone-0.10
titlemod_mam: do not enable MAM by default on all accounts MAM: Enable archiving after users first query
Zash
on
MattJ was opposed.
Changes
tags Status-New low
Zash
on
...
Changes
tagslow Priority-Low
Zash
on
Unlikely to happen in 0.10, however it could probably me made as a plugin.
Changes
tagsMilestone-0.10
Ge0rG
on
Just to reiterate that this is also a data privacy problem. MAM is auto-enabled for users without any opt-in. A MAM-enabled client at least in theory could inform the user about activating server-side storage.
Ge0rG
on
And to document the admin console "workaround", which I ended up looking for in the prosody MUC logs multiple times now, which disables archive for individual users:
>require"core.storagemanager".open("SERVER", "archive_prefs", "map"):set("USER", false, false)
MattJ
on
Convinced by Ge0rG that a user without a MAM client has no way to opt-out, a user with a MAM client may.
Changes
tags Status-Accepted
Zash
on
Should this go in 0.10 or trunk?
Changes
tags Status-Started
Ge0rG
on
Please add it to 0.10; I'd be okay with it being off-by-default to not break compatibility. Probably this setting mostly matters for public servers only anyway (due to privacy laws), but I don't want you to make it depend on whether IBR is enabled.
Once mod_mam is loaded, it will log data for all users on the server, even for high-traffic users who never actually use MAM. Please copy the behavior of ejabberd where MAM is only enabled for a given account once that user accesses the MAM archive once.
mod_mam: do not enable MAM by default on all accountsMAM: Enable archiving after users first queryMattJ was opposed.
Changes...
ChangeslowPriority-LowUnlikely to happen in 0.10, however it could probably me made as a plugin.
ChangesMilestone-0.10Just to reiterate that this is also a data privacy problem. MAM is auto-enabled for users without any opt-in. A MAM-enabled client at least in theory could inform the user about activating server-side storage.
And to document the admin console "workaround", which I ended up looking for in the prosody MUC logs multiple times now, which disables archive for individual users: >require"core.storagemanager".open("SERVER", "archive_prefs", "map"):set("USER", false, false)
Convinced by Ge0rG that a user without a MAM client has no way to opt-out, a user with a MAM client may.
ChangesShould this go in 0.10 or trunk?
ChangesPlease add it to 0.10; I'd be okay with it being off-by-default to not break compatibility. Probably this setting mostly matters for public servers only anyway (due to privacy laws), but I don't want you to make it depend on whether IBR is enabled.
Done in https://hg.prosody.im/0.10/rev/3eb4cafb3b64
Changes