Software /
code /
prosody
Changeset
8654:425de10efde4
storagemanager: Log warning on storage access outside of async contexts
author | Matthew Wild <mwild1@gmail.com> |
---|---|
date | Thu, 22 Mar 2018 16:25:20 +0000 |
parents | 8653:dab9b8af0f01 |
children | 8655:ba6a6a04b46c |
files | core/storagemanager.lua |
diffstat | 1 files changed, 32 insertions(+), 0 deletions(-) [+] |
line wrap: on
line diff
--- a/core/storagemanager.lua Thu Mar 22 16:24:22 2018 +0000 +++ b/core/storagemanager.lua Thu Mar 22 16:25:20 2018 +0000 @@ -1,6 +1,7 @@ local type, pairs = type, pairs; local setmetatable = setmetatable; +local rawset, error = rawset, error; local config = require "core.configmanager"; local datamanager = require "util.datamanager"; @@ -8,6 +9,8 @@ local multitable = require "util.multitable"; local hosts = hosts; local log = require "util.logger".init("storagemanager"); +local async = require "util.async"; +local debug = debug; local prosody = prosody; @@ -29,8 +32,34 @@ } ); +local async_check = config.get("*", "storage_async_check") ~= false; + local stores_available = multitable.new(); +local function check_async_wrapper(event) + local store = event.store; + event.store = setmetatable({}, { + __index = function (t, method_name) + local original_method = store[method_name]; + if type(original_method) ~= "function" then + if original_method then + rawset(t, method_name, original_method); + end + return original_method; + end + local wrapped_method = function (...) + if not async.ready() then + log("warn", "ASYNC-01: Attempt to access storage outside async context, " + .."see https://prosody.im/doc/developers/async - %s", debug.traceback()); + end + return original_method(...); + end + rawset(t, method_name, wrapped_method); + return wrapped_method; + end; + }); +end + local function initialize_host(host) local host_session = hosts[host]; host_session.events.add_handler("item-added/storage-provider", function (event) @@ -42,6 +71,9 @@ local item = event.item; stores_available:set(host, item.name, nil); end); + if async_check then + host_session.events.add_handler("store-opened", check_async_wrapper); + end end prosody.events.add_handler("host-activated", initialize_host, 101);