Systemd log format activable via env var

[RCasatta]

I updated my program with something along the lines of https://github.com/rust-cli/env_logger/blob/main/examples/syslog_friendly_format.rs to have a systemd friendly logging, however when I use other binaries using env_logger usually the same logic is not available, can we have it enable-able by default in env_logger?

ie if RUST_LOG_STYLE=SYSTEMD the default formatting is changed with the systemd friendly one without having to change the downstream lib

[djugei]

specifically for systemd its quite easy to automatically detect being run as a daemon.
i am sure that its possible for other service managers/init systems too, but to the best of my knowledge systemd simply is the main one by far.

[epage]

@djugei in #357, you also mentioned the idea of having a separate format string.

What value would users get by having all of this baked into env_logger::Logger vs having a way to compose multiple loggers, including a syslog logger?

[djugei]

defaults matter a lot. developers want to run their binaries in terminal and see timestamps, users want to run applications as daemons and see timestamps only once. people simply will not have a more complex setup.

and this is a real issue, i am sure @RCasatta has observed this in a rust application they were running, and so have i.
ofc if the app is open source we can go around and make prs for every single project.
or just one here.
the later seems like the smarter option.

[epage]

Sorry, I missed read this (being used to multi-platform system logs being rendered out of band from stdout). Looking back over #358, this is just about changing the output format.

env_logger then auto-detects if its running as a daemon

I am open to auto-detection.

env_logger then auto-detects if its running as a daemon, with an override available.

I don't think we need a specific variable to control the auto-detection: we can expose a function that matches the custom format signature and people can customize things as they want.

ie if RUST_LOG_STYLE=SYSTEMD the default formatting is changed with the systemd friendly one without having to change the downstream lib

That is specific to binaries, not libraries, and I assume binaries will generally know they will be run in a daemon mode to support something like this.

#358

The format used in this was

                Box::new(|buf, record| {
                    writeln!(
                        buf,
                        "<{}>{}: {}",
                        match record.level() {
                            Level::Error => 3,
                            Level::Warn => 4,
                            Level::Info => 6,
                            Level::Debug => 7,
                            Level::Trace => 7,
                        },
                        record.target(),
                        record.args()
                    )
                })

Where is the relevant documentation for this? My initial searches come up short.

[djugei]

the format was literally copied from the example in this repository, after cross-checking with the relevant manpages to verify that this is actually a valid way to log to syslog. there is more valid ways, for example by providing a timestamp, but that gets auto-filled in by the syslog.
syslog format is.. semi-standardtized but that setup seems to work for journald.

[RCasatta]

I confirm I use it with systemd and it works. For example you can filter error logs via:

journalctl -u unit -p 3

[epage]

the format was literally copied from the example in this repository, after cross-checking with the relevant manpages to verify that this is actually a valid way to log to syslog. there is more valid ways, for example by providing a timestamp, but that gets auto-filled in by the syslog.
syslog format is.. semi-standardtized but that setup seems to work for journald.

Was not aware of that example, thanks!

In that case, for anything we do, we should update or remove that example. We'll also need to ensure people have a good way of discovering this functionality.

[djugei]

I confirm I use it with systemd and it works. For example you can filter error logs via:

journalctl -u unit -p 3

just to be explicit here: you have tested #358 and it solves your usecase?

[djugei]

In that case, for anything we do, we should update or remove that example. We'll also need to ensure people have a good way of discovering this functionality.

i have updated the example. discoverability is not worse now, and also somewhat optional as it should just work, thats the magic of good defaults.
as mentioned in #358 (comment) please do advise where adding documentation would be useful.

[RCasatta]

I confirm I use it with systemd and it works. For example you can filter error logs via:

journalctl -u unit -p 3

just to be explicit here: you have tested #358 and it solves your usecase?

I was referring to the log format, I didn't test with #358

[epage]

@djugei what are your thoughts on the plain I laid out in #349 (comment)?

To re-summarize

• Auto-detect we are running as a service and switch the format
• Provide a syslog_format and ConfigurableFormat, allowing people to remove the auto-detection by passing ConfigurableFormat as a custom format, or providing their own auto-detection by wrapping syslog_format and ConfigurableFormat

The main question I have is whether we provide a syslog_format function or a SysLogFormat struct that then allows customization in the future.