Merge 0.10->trunk
authorMatthew Wild <mwild1@gmail.com>
Fri, 5 Feb 2016 00:04:11 +0000 (00:04 +0000)
committerMatthew Wild <mwild1@gmail.com>
Fri, 5 Feb 2016 00:04:11 +0000 (00:04 +0000)
core/certmanager.lua
core/loggingmanager.lua
core/moduleapi.lua
plugins/mod_groups.lua
util/logger.lua

index db3cf58e296f21c86b9f299b68b7ce5a8a8708e2..b1ff648dab7f91c6f8f31c3e47653a180d6143ba 100644 (file)
@@ -56,12 +56,11 @@ local global_certificates = configmanager.get("*", "certificates") or "certs";
 local crt_try = { "", "/%s.crt", "/%s/fullchain.pem", "/%s.pem", };
 local key_try = { "", "/%s.key", "/%s/privkey.pem",   "/%s.pem", };
 
-local function find_cert(host)
-       local certs = configmanager.get(host, "certificate") or global_certificates;
-       certs = resolve_path(config_path, certs);
+local function find_cert(user_certs, name)
+       local certs = resolve_path(config_path, user_certs or global_certificates);
        for i = 1, #crt_try do
-               local crt_path = certs .. crt_try[i]:format(host);
-               local key_path = certs .. key_try[i]:format(host);
+               local crt_path = certs .. crt_try[i]:format(name);
+               local key_path = certs .. key_try[i]:format(name);
 
                if stat(crt_path, "mode") == "file" then
                        if stat(key_path, "mode") == "file" then
@@ -77,6 +76,19 @@ local function find_cert(host)
        end
 end
 
+local function find_host_cert(host)
+       if not host then return nil; end
+       return find_cert(configmanager.get(host, "certificate"), host) or find_host_cert(host:match("%.(.+)$"));
+end
+
+local function find_service_cert(service, port)
+       local cert_config = configmanager.get("*", service.."_certificate");
+       if type(cert_config) == "table" then
+               cert_config = cert_config[port] or cert_config.default;
+       end
+       return find_cert(cert_config, service);
+end
+
 -- Built-in defaults
 local core_defaults = {
        capath = "/etc/ssl/certs";
@@ -109,7 +121,12 @@ local function create_context(host, mode, ...)
        local cfg = new_config();
        cfg:apply(core_defaults);
        cfg:apply(global_ssl_config);
-       cfg:apply(find_cert(host) or find_cert(host:match("%.(.*)")));
+       local service_name, port = host:match("^(%w+) port (%d+)$");
+       if service_name then
+               cfg:apply(find_service_cert(service_name, tonumber(port)));
+       else
+               cfg:apply(find_host_cert(host));
+       end
        cfg:apply({
                mode = mode,
                -- We can't read the password interactively when daemonized
index 259c2c4411eca9bf71aed8355dec730f7498a856..77d31964d0d3545b321ac18799e2be396081767a 100644 (file)
 local format = string.format;
 local setmetatable, rawset, pairs, ipairs, type =
        setmetatable, rawset, pairs, ipairs, type;
-local io_open, io_write = io.open, io.write;
+local stdout = io.stdout;
+local io_open = io.open;
 local math_max, rep = math.max, string.rep;
 local os_date = os.date;
-local getstyle, setstyle = require "util.termcolours".getstyle, require "util.termcolours".setstyle;
-
--- COMPAT: This should no longer be needed since the addition of setvbuf calls
-if os.getenv("__FLUSH_LOG") then
-       local io_flush = io.flush;
-       local _io_write = io_write;
-       io_write = function(...) _io_write(...); io_flush(); end
-end
+local getstyle, getstring = require "util.termcolours".getstyle, require "util.termcolours".getstring;
+local tostring = tostring;
+local unpack = table.unpack or unpack;
 
 local config = require "core.configmanager";
 local logger = require "util.logger";
@@ -34,7 +30,7 @@ local _ENV = nil;
 -- The log config used if none specified in the config file (see reload_logging for initialization)
 local default_logging;
 local default_file_logging;
-local default_timestamp = "%b %d %H:%M:%S";
+local default_timestamp = "%b %d %H:%M:%S ";
 -- The actual config loggingmanager is using
 local logging_config;
 
@@ -154,7 +150,6 @@ local function reload_logging()
        default_file_logging = {
                { to = "file", levels = { min = (debug_mode and "debug") or "info" }, timestamps = true }
        };
-       default_timestamp = "%b %d %H:%M:%S";
 
        logging_config = config.get("*", "log") or default_logging;
 
@@ -171,114 +166,90 @@ prosody.events.add_handler("config-reloaded", reload_logging);
 --- Definition of built-in logging sinks ---
 
 -- Null sink, must enter log_sink_types *first*
-function log_sink_types.nowhere()
+local function log_to_nowhere()
        return function () return false; end;
 end
+log_sink_types.nowhere = log_to_nowhere;
 
--- Column width for "source" (used by stdout and console)
-local sourcewidth = 20;
+local function log_to_file(sink_config, logfile)
+       logfile = logfile or io_open(sink_config.filename, "a+");
+       if not logfile then
+               return log_to_nowhere(sink_config);
+       end
+       local write = logfile.write;
 
-function log_sink_types.stdout(sink_config)
        local timestamps = sink_config.timestamps;
 
        if timestamps == true then
                timestamps = default_timestamp; -- Default format
+       elseif timestamps then
+               timestamps = timestamps .. " ";
        end
 
        if sink_config.buffer_mode ~= false then
-               io.stdout:setvbuf(sink_config.buffer_mode or "line");
-       end
-
-       return function (name, level, message, ...)
-               sourcewidth = math_max(#name+2, sourcewidth);
-               local namelen = #name;
-               if timestamps then
-                       io_write(os_date(timestamps), " ");
-               end
-               if ... then
-                       io_write(name, rep(" ", sourcewidth-namelen), level, "\t", format(message, ...), "\n");
-               else
-                       io_write(name, rep(" ", sourcewidth-namelen), level, "\t", message, "\n");
-               end
-       end
-end
-
-do
-       local do_pretty_printing = true;
-
-       local logstyles = {};
-       if do_pretty_printing then
-               logstyles["info"] = getstyle("bold");
-               logstyles["warn"] = getstyle("bold", "yellow");
-               logstyles["error"] = getstyle("bold", "red");
+               logfile:setvbuf(sink_config.buffer_mode or "line");
        end
-       function log_sink_types.console(sink_config)
-               -- Really if we don't want pretty colours then just use plain stdout
-               if not do_pretty_printing then
-                       return log_sink_types.stdout(sink_config);
-               end
 
-               local timestamps = sink_config.timestamps;
-
-               if timestamps == true then
-                       timestamps = default_timestamp; -- Default format
-               end
+       -- Column width for "source" (used by stdout and console)
+       local sourcewidth = sink_config.source_width;
 
-               if sink_config.buffer_mode ~= false then
-                       io.stdout:setvbuf(sink_config.buffer_mode or "line");
+       return function (name, level, message, ...)
+               local n = select('#', ...);
+               if n ~= 0 then
+                       local arg = { ... };
+                       for i = 1, n do
+                               arg[i] = tostring(arg[i]);
+                       end
+                       message = format(message, unpack(arg, 1, n));
                end
 
-               return function (name, level, message, ...)
+               if sourcewidth then
                        sourcewidth = math_max(#name+2, sourcewidth);
-                       local namelen = #name;
-
-                       if timestamps then
-                               io_write(os_date(timestamps), " ");
-                       end
-                       io_write(name, rep(" ", sourcewidth-namelen));
-                       setstyle(logstyles[level]);
-                       io_write(level);
-                       setstyle();
-                       if ... then
-                               io_write("\t", format(message, ...), "\n");
-                       else
-                               io_write("\t", message, "\n");
-                       end
+                       name = name ..  rep(" ", sourcewidth-#name);
+               else
+                       name = name .. "\t";
                end
+               write(logfile, timestamps and os_date(timestamps) or "", name, level, "\t", message, "\n");
        end
 end
+log_sink_types.file = log_to_file;
 
-local empty_function = function () end;
-function log_sink_types.file(sink_config)
-       local log = sink_config.filename;
-       local logfile = io_open(log, "a+");
-       if not logfile then
-               return empty_function;
+local function log_to_stdout(sink_config)
+       if not sink_config.timestamps then
+               sink_config.timestamps = false;
        end
-
-       if sink_config.buffer_mode ~= false then
-               logfile:setvbuf(sink_config.buffer_mode or "line");
+       if sink_config.source_width == nil then
+               sink_config.source_width = 20;
        end
+       return log_to_file(sink_config, stdout);
+end
+log_sink_types.stdout = log_to_stdout;
 
-       local write = logfile.write;
+local do_pretty_printing = true;
 
-       local timestamps = sink_config.timestamps;
+local logstyles;
+if do_pretty_printing then
+       logstyles = {};
+       logstyles["info"] = getstyle("bold");
+       logstyles["warn"] = getstyle("bold", "yellow");
+       logstyles["error"] = getstyle("bold", "red");
+end
 
-       if timestamps == nil or timestamps == true then
-               timestamps = default_timestamp; -- Default format
+local function log_to_console(sink_config)
+       -- Really if we don't want pretty colours then just use plain stdout
+       local logstdout = log_to_stdout(sink_config);
+       if not do_pretty_printing then
+               return logstdout;
        end
-
        return function (name, level, message, ...)
-               if timestamps then
-                       write(logfile, os_date(timestamps), " ");
-               end
-               if ... then
-                       write(logfile, name, "\t", level, "\t", format(message, ...), "\n");
-               else
-                       write(logfile, name, "\t" , level, "\t", message, "\n");
+               local logstyle = logstyles[level];
+               if logstyle then
+                       level = getstring(logstyle, level);
                end
-       end;
+               return logstdout(name, level, message, ...);
+       end
 end
+log_sink_types.console = log_to_console;
 
 local function register_sink_type(name, sink_maker)
        local old_sink_maker = log_sink_types[name];
index bdf9959f7d098de8db7fc88fcc31e0d23aa378df..ff68a15ce420dce6e25dd07e6616cd97e6fe41aa 100644 (file)
@@ -303,6 +303,20 @@ function api:get_option_inherited_set(name, ...)
        return value;
 end
 
+function api:get_option_path(name, default, parent)
+       if parent == nil then
+               parent = parent or self:get_directory();
+       elseif prosody.paths[parent] then
+               parent = prosody.paths[parent];
+       end
+       local value = self:get_option_string(name, default);
+       if value == nil then
+               return nil;
+       end
+       return resolve_relative_path(parent, value);
+end
+
+
 function api:context(host)
        return setmetatable({host=host or "*"}, {__index=self,__newindex=self});
 end
index be1a5508f87021d24a1c8d064060ba667f711f38..d696d45388df400f49caa168de6423ae314589d7 100644 (file)
@@ -10,8 +10,6 @@
 local groups;
 local members;
 
-local groups_file;
-
 local jid, datamanager = require "util.jid", require "util.datamanager";
 local jid_prep = jid.prep;
 
@@ -82,7 +80,7 @@ function remove_virtual_contacts(username, host, datastore, data)
 end
 
 function module.load()
-       groups_file = module:get_option_string("groups_file");
+       local groups_file = module:get_option_path("groups_file", nil, "config");
        if not groups_file then return; end
 
        module:hook("roster-load", inject_roster_contacts);
index 3d1f1c8bffc7c7aa971c63db10feda513819e5d2..e72b29bc05843987867a66c42526fefd1bb2527f 100644 (file)
@@ -5,11 +5,9 @@
 -- This project is MIT/X11 licensed. Please see the
 -- COPYING file in the source package for more information.
 --
+-- luacheck: ignore 213/level
 
-local pcall = pcall;
-
-local find = string.find;
-local ipairs, pairs, setmetatable = ipairs, pairs, setmetatable;
+local pairs = pairs;
 
 local _ENV = nil;