util.xmppstream: Send <?xml?> declaration in same call as stream open If network_settings.opportunistic_writes is enabled then this would previously have resulted in two socket writes, and possibly two packets being sent. This caused some issues in older versions of Gajim, which apparently expected the stream opening in the first packet, and thus it could not connect. With this change and opportunistic_writes enabled, the first packet should contain both the xml declaration and the stream open tag. Without opportunistic_writes, there should be no observable change. Tested with Gajim 1.1.2 (on same machine). Unsure if loopback behaves differently than the network here.

core.statsmanager: Allow special "manual" value for statistics_interval When set, no periodic statistics collection is done by core.statsmanager, instead some module is expected to call collect() when it suits. Obviously only one such module should be enabled. Quoth jonas’ > correct way is to scrape the internal sources on each call to /metrics > in the context of Prometheus "manual" as opposed to "automatic", from the point of view of statsmanager.

Merge 0.11->trunk

mod_saslauth: Use a defined SASL error

mod_saslauth: Improve code style This many returns deserve their own line. `session["sasl_handler"]` style isn't used anywhere else.

mod_c2s: Log about missing conn on async state changes

mod_c2s: Improve code style We don't use the quoted table indexing style that often, it's not needed here and it's enough to check for falsyness rather than `nil`.

mod_c2s: Fix traceback in session close when conn is nil Unclear how this happens.

mod_saslauth: Don't throw errors in async code when connections are gone Fixes #1515

mod_c2s: Don't throw errors in async code when connections are gone Fixes #1507