badlop

Hi! Now that the other MIX PRs are merged, can you rebase this one and fix the merge conflict? This will also allow CI to run again, and probably pass...

Thanks, merged now. It seems I didn't get a notification...

The minimal implementation could be something like this, in the yconf library that ejabberd uses to parse the yaml configuration files: ```diff diff --git a/src/yconf.erl b/src/yconf.erl index 8967067..d93868d 100644 ---...

For testing purposes, if you can't apply the patch to yconf, compile yconf and compile ejabberd, there's a simple solution: use this container image https://github.com/badlop/ejabberd/pkgs/container/ejabberd/24688553?tag=3804 which includes patched yconf and...

As explained by Holger in the chatroom: > I submitted them as PRs rather than just pushing the changes because they involve trade-offs and I don't want to make any...

The 'roster' record includes a 'xs' field, I imagine it was there in case we want to store extra information like this. Mnesia stores the 'roster' record as is, so...

Looking at the source code, the opposite `mnesia_down` event is handled since years ago. The most recent change in that code was in 2017: https://github.com/processone/ejabberd/commit/8639da0e00b0949b238cfbbf30845ec21976003c The `warning unexpected info` was...

Quick patch to send to journald: ```diff From 5ff3a4f14d613eef0a9c19e7d56f42a3cd498cc4 Mon Sep 17 00:00:00 2001 From: Badlop Date: Wed, 25 May 2022 03:10:04 +0200 Subject: [PATCH] Add logger_journald library and send...

> Currently I am parsing the eJabberd logs in realtime and preparing logs in structured format compatible with Logstash. Did you check if there are libraries that already provide that...

I pushed this commit to my fork repository. It works for me: https://github.com/badlop/ejabberd/commit/8c0da87012a9f72f2e869e93b733bead168febb5 I just learned about logstash, so you may need to update configuration, etc for a real production...