util.datamapper: Fix error on attempt to coerce nil to something Turns falsy values into nil instead of nothing, which ensures this function always has 1 return value, or table.insert({}) complains. Would still happen on some unexpected input, but that's actually a good thing.

mod_admin_shell: Sort timers by time in debug:timers() Easier to see which timers are happening soon vs further in the future if they are in some sensible order.

mod_c2s: Log a debug message before closing due to c2s_timeout It was confusing that the connection would just close without much explanation. Wanted this while investigating https://github.com/conversejs/converse.js/issues/2438

util.datamapper: Deal with locally built stanzas missing xmlns So the problem is that xmlns is not inherited when building a stanza, and then :get_child(n, ns) with an explicit namespace does not find that such child tags. E.g. local t = st.stanza("foo", { xmlns = "urn:example:bar" }) :text_tag("hello", "world"); assert(t:get_child("hello", "urn:example:bar"), "This fails"); Meanwhile, during parsing (util.xmppstream or util.xml) child tags do get the parents xmlns when not overriding them. Thus, in the above example, if the stanza is passed trough `t = util.xml.parse(tostring(t))` then the assert succeeds. This change makes it so that it leaves out the namespace argument to :get_child when it is the same as the current/parent namespace, which behaves the same for both built and parsed stanzas.

util.datamapper: Factor out conversion from any value to XML string Since this was the last severely duplicated code left.

mod_authz_internal: Ignore unused argument for now [luachec]

usermanager: expose set_roles through API

mod_authz_internal: add support for setting roles of a local user

util.datamapper: Handle nested arrays or objects in arrays

util.datamapper: Fix to skip parsing wrapped arrays that aren't there Turns out the unreachable error is reachable :D