util.datamanager: Import tostring and type (fix global access)
[prosody.git] / util / datamanager.lua
index 23a2d74f187ef24b697107dc187ed9eae06c9ab8..fb9ba3a4ce6283f6eb33abce53a218f692cc96c6 100644 (file)
@@ -8,31 +8,44 @@
 
 
 local format = string.format;
-local setmetatable, type = setmetatable, type;
-local pairs, ipairs = pairs, ipairs;
+local setmetatable = setmetatable;
+local ipairs = ipairs;
 local char = string.char;
 local pcall = pcall;
 local log = require "util.logger".init("datamanager");
 local io_open = io.open;
 local os_remove = os.remove;
-local tostring, tonumber = tostring, tonumber;
-local error = error;
+local os_rename = os.rename;
+local tonumber = tonumber;
+local tostring = tostring;
 local next = next;
+local type = type;
 local t_insert = table.insert;
-local append = require "util.serialization".append;
+local t_concat = table.concat;
 local envloadfile = require"util.envload".envloadfile;
+local serialize = require "util.serialization".serialize;
 local path_separator = assert ( package.config:match ( "^([^\n]+)" ) , "package.config not in standard form" ) -- Extract directory seperator from package.config (an undocumented string that comes with lua)
 local lfs = require "lfs";
 local prosody = prosody;
-local raw_mkdir;
 
-if prosody.platform == "posix" then
-       raw_mkdir = require "util.pposix".mkdir; -- Doesn't trample on umask
-else
-       raw_mkdir = lfs.mkdir;
-end
+local raw_mkdir = lfs.mkdir;
+local function fallocate(f, offset, len)
+       -- This assumes that current position == offset
+       local fake_data = (" "):rep(len);
+       local ok, msg = f:write(fake_data);
+       if not ok then
+               return ok, msg;
+       end
+       f:seek("set", offset);
+       return true;
+end;
+pcall(function()
+       local pposix = require "util.pposix";
+       raw_mkdir = pposix.mkdir or raw_mkdir; -- Doesn't trample on umask
+       fallocate = pposix.fallocate or fallocate;
+end);
 
-module "datamanager"
+local _ENV = nil;
 
 ---- utils -----
 local encode, decode;
@@ -63,7 +76,7 @@ local callbacks = {};
 
 ------- API -------------
 
-function set_data_path(path)
+local function set_data_path(path)
        log("debug", "Setting data path to: %s", path);
        data_path = path;
 end
@@ -76,14 +89,14 @@ local function callback(username, host, datastore, data)
 
        return username, host, datastore, data;
 end
-function add_callback(func)
+local function add_callback(func)
        if not callbacks[func] then -- Would you really want to set the same callback more than once?
                callbacks[func] = true;
                callbacks[#callbacks+1] = func;
                return true;
        end
 end
-function remove_callback(func)
+local function remove_callback(func)
        if callbacks[func] then
                for i, f in ipairs(callbacks) do
                        if f == func then
@@ -95,23 +108,20 @@ function remove_callback(func)
        end
 end
 
-function getpath(username, host, datastore, ext, create)
+local function getpath(username, host, datastore, ext, create)
        ext = ext or "dat";
        host = (host and encode(host)) or "_global";
        username = username and encode(username);
        if username then
                if create then mkdir(mkdir(mkdir(data_path).."/"..host).."/"..datastore); end
                return format("%s/%s/%s/%s.%s", data_path, host, datastore, username, ext);
-       elseif host then
+       else
                if create then mkdir(mkdir(data_path).."/"..host); end
                return format("%s/%s/%s.%s", data_path, host, datastore, ext);
-       else
-               if create then mkdir(data_path); end
-               return format("%s/%s.%s", data_path, datastore, ext);
        end
 end
 
-function load(username, host, datastore)
+local function load(username, host, datastore)
        local data, ret = envloadfile(getpath(username, host, datastore), {});
        if not data then
                local mode = lfs.attributes(getpath(username, host, datastore), "mode");
@@ -133,7 +143,44 @@ function load(username, host, datastore)
        return ret;
 end
 
-function store(username, host, datastore, data)
+local function atomic_store(filename, data)
+       local scratch = filename.."~";
+       local f, ok, msg;
+
+       f, msg = io_open(scratch, "w");
+       if not f then
+               return nil, msg;
+       end
+
+       ok, msg = f:write(data);
+       if not ok then
+               f:close();
+               os_remove(scratch);
+               return nil, msg;
+       end
+
+       ok, msg = f:close();
+       if not ok then
+               os_remove(scratch);
+               return nil, msg;
+       end
+
+       return os_rename(scratch, filename);
+end
+
+if prosody and prosody.platform ~= "posix" then
+       -- os.rename does not overwrite existing files on Windows
+       -- TODO We could use Transactional NTFS on Vista and above
+       function atomic_store(filename, data)
+               local f, err = io_open(filename, "w");
+               if not f then return f, err; end
+               local ok, msg = f:write(data);
+               if not ok then f:close(); return ok, msg; end
+               return f:close();
+       end
+end
+
+local function store(username, host, datastore, data)
        if not data then
                data = {};
        end
@@ -144,56 +191,99 @@ function store(username, host, datastore, data)
        end
 
        -- save the datastore
-       local f, msg = io_open(getpath(username, host, datastore, nil, true), "w+");
+       local d = "return " .. serialize(data) .. ";\n";
+       local mkdir_cache_cleared;
+       repeat
+               local ok, msg = atomic_store(getpath(username, host, datastore, nil, true), d);
+               if not ok then
+                       if not mkdir_cache_cleared then -- We may need to recreate a removed directory
+                               _mkdir = {};
+                               mkdir_cache_cleared = true;
+                       else
+                               log("error", "Unable to write to %s storage ('%s') for user: %s@%s", datastore, msg, username or "nil", host or "nil");
+                               return nil, "Error saving to storage";
+                       end
+               end
+               if next(data) == nil then -- try to delete empty datastore
+                       log("debug", "Removing empty %s datastore for user %s@%s", datastore, username or "nil", host or "nil");
+                       os_remove(getpath(username, host, datastore));
+               end
+               -- we write data even when we are deleting because lua doesn't have a
+               -- platform independent way of checking for non-exisitng files
+       until ok;
+       return true;
+end
+
+-- Append a blob of data to a file
+local function append(username, host, datastore, ext, data)
+       if type(data) ~= "string" then return; end
+       local filename = getpath(username, host, datastore, ext, true);
+
+       local ok;
+       local f, msg = io_open(filename, "r+");
        if not f then
-               log("error", "Unable to write to %s storage ('%s') for user: %s@%s", datastore, msg, username or "nil", host or "nil");
-               return nil, "Error saving to storage";
+               -- File did probably not exist, let's create it
+               f, msg = io_open(filename, "w");
+               if not f then
+                       return nil, msg, "open";
+               end
        end
-       f:write("return ");
-       append(f, data);
-       f:close();
-       if next(data) == nil then -- try to delete empty datastore
-               log("debug", "Removing empty %s datastore for user %s@%s", datastore, username or "nil", host or "nil");
-               os_remove(getpath(username, host, datastore));
+
+       local pos = f:seek("end");
+       ok, msg = fallocate(f, pos, #data);
+       if not ok then
+               log("warn", "fallocate() failed: %s", tostring(msg));
+               -- This doesn't work on every file system
        end
-       -- we write data even when we are deleting because lua doesn't have a
-       -- platform independent way of checking for non-exisitng files
-       return true;
+
+       if f:seek() ~= pos then
+               log("debug", "fallocate() changed file position");
+               f:seek("set", pos);
+       end
+
+       ok, msg = f:write(data);
+       if not ok then
+               f:close();
+               return ok, msg, "write";
+       end
+
+       ok, msg = f:close();
+       if not ok then
+               return ok, msg;
+       end
+
+       return true, pos;
 end
 
-function list_append(username, host, datastore, data)
+local function list_append(username, host, datastore, data)
        if not data then return; end
        if callback(username, host, datastore) == false then return true; end
        -- save the datastore
-       local f, msg = io_open(getpath(username, host, datastore, "list", true), "a+");
-       if not f then
+
+       data = "item(" ..  serialize(data) .. ");\n";
+       local ok, msg = append(username, host, datastore, "list", data);
+       if not ok then
                log("error", "Unable to write to %s storage ('%s') for user: %s@%s", datastore, msg, username or "nil", host or "nil");
-               return;
+               return ok, msg;
        end
-       f:write("item(");
-       append(f, data);
-       f:write(");\n");
-       f:close();
        return true;
 end
 
-function list_store(username, host, datastore, data)
+local function list_store(username, host, datastore, data)
        if not data then
                data = {};
        end
        if callback(username, host, datastore) == false then return true; end
        -- save the datastore
-       local f, msg = io_open(getpath(username, host, datastore, "list", true), "w+");
-       if not f then
+       local d = {};
+       for _, item in ipairs(data) do
+               d[#d+1] = "item(" .. serialize(item) .. ");\n";
+       end
+       local ok, msg = atomic_store(getpath(username, host, datastore, "list", true), t_concat(d));
+       if not ok then
                log("error", "Unable to write to %s storage ('%s') for user: %s@%s", datastore, msg, username or "nil", host or "nil");
                return;
        end
-       for _, d in ipairs(data) do
-               f:write("item(");
-               append(f, d);
-               f:write(");\n");
-       end
-       f:close();
        if next(data) == nil then -- try to delete empty datastore
                log("debug", "Removing empty %s datastore for user %s@%s", datastore, username or "nil", host or "nil");
                os_remove(getpath(username, host, datastore, "list"));
@@ -203,7 +293,7 @@ function list_store(username, host, datastore, data)
        return true;
 end
 
-function list_load(username, host, datastore)
+local function list_load(username, host, datastore)
        local items = {};
        local data, ret = envloadfile(getpath(username, host, datastore, "list"), {item = function(i) t_insert(items, i); end});
        if not data then
@@ -226,45 +316,104 @@ function list_load(username, host, datastore)
        return items;
 end
 
-function list_stores(username, host)
-       if not host then
-               return nil, "bad argument #2 to 'list_stores' (string expected, got nothing)";
+local type_map = {
+       keyval = "dat";
+       list = "list";
+}
+
+local function users(host, store, typ)
+       typ = type_map[typ or "keyval"];
+       local store_dir = format("%s/%s/%s", data_path, encode(host), store);
+
+       local mode, err = lfs.attributes(store_dir, "mode");
+       if not mode then
+               return function() log("debug", "%s", err or (store_dir .. " does not exist")) end
        end
-       local list = {};
-       local host_dir = format("%s/%s/", data_path, encode(host));
-       for node in lfs.dir(host_dir) do
-               if not node:match"^%." then -- dots should be encoded, this is probably . or ..
-                       local store = decode(node);
-                       local path = host_dir..node;
-                       if username == true then
-                               if lfs.attributes(path, "mode") == "directory" then
-                                       list[#list+1] = store;
-                               end
-                       elseif username then
-                               if lfs.attributes(getpath(username, host, store), "mode")
-                                       or lfs.attributes(getpath(username, host, store, "list"), "mode") then
-                                       list[#list+1] = store;
+       local next, state = lfs.dir(store_dir);
+       return function(state)
+               for node in next, state do
+                       local file, ext = node:match("^(.*)%.([dalist]+)$");
+                       if file and ext == typ then
+                               return decode(file);
+                       end
+               end
+       end, state;
+end
+
+local function stores(username, host, typ)
+       typ = type_map[typ or "keyval"];
+       local store_dir = format("%s/%s/", data_path, encode(host));
+
+       local mode, err = lfs.attributes(store_dir, "mode");
+       if not mode then
+               return function() log("debug", err or (store_dir .. " does not exist")) end
+       end
+       local next, state = lfs.dir(store_dir);
+       return function(state)
+               for node in next, state do
+                       if not node:match"^%." then
+                               if username == true then
+                                       if lfs.attributes(store_dir..node, "mode") == "directory" then
+                                               return decode(node);
+                                       end
+                               elseif username then
+                                       local store = decode(node)
+                                       if lfs.attributes(getpath(username, host, store, typ), "mode") then
+                                               return store;
+                                       end
+                               elseif lfs.attributes(node, "mode") == "file" then
+                                       local file, ext = node:match("^(.*)%.([dalist]+)$");
+                                       if ext == typ then
+                                               return decode(file)
+                                       end
                                end
-                       elseif lfs.attributes(path, "mode") == "file" then
-                               list[#list+1] = store:gsub("%.[dalist]+$","");
                        end
                end
+       end, state;
+end
+
+local function do_remove(path)
+       local ok, err = os_remove(path);
+       if not ok and lfs.attributes(path, "mode") then
+               return ok, err;
        end
-       return list;
+       return true
 end
 
-function purge(username, host)
+local function purge(username, host)
        local host_dir = format("%s/%s/", data_path, encode(host));
-       local deleted = 0;
-       for file in lfs.dir(host_dir) do
+       local ok, iter, state, var = pcall(lfs.dir, host_dir);
+       if not ok then
+               return ok, iter;
+       end
+       local errs = {};
+       for file in iter, state, var do
                if lfs.attributes(host_dir..file, "mode") == "directory" then
                        local store = decode(file);
-                       deleted = deleted + (os_remove(getpath(username, host, store)) and 1 or 0);
-                       deleted = deleted + (os_remove(getpath(username, host, store, "list")) and 1 or 0);
-                       -- We this will generate loads of "No such file or directory", but do we care?
+                       local ok, err = do_remove(getpath(username, host, store));
+                       if not ok then errs[#errs+1] = err; end
+
+                       local ok, err = do_remove(getpath(username, host, store, "list"));
+                       if not ok then errs[#errs+1] = err; end
                end
        end
-       return deleted > 0, deleted;
+       return #errs == 0, t_concat(errs, ", ");
 end
 
-return _M;
+return {
+       set_data_path = set_data_path;
+       add_callback = add_callback;
+       remove_callback = remove_callback;
+       getpath = getpath;
+       load = load;
+       store = store;
+       append_raw = append;
+       list_append = list_append;
+       list_store = list_store;
+       list_load = list_load;
+       users = users;
+       stores = stores;
+       purge = purge;
+       path_decode = decode;
+       path_encode = encode;
+};