env_logger icon indicating copy to clipboard operation
env_logger copied to clipboard

feat: Syslog-adapted formating and auto-detection

Open djugei opened this issue 9 months ago • 7 comments

closes #357 closes #349

this is a surprisingly low diff size.

djugei avatar Mar 30 '25 18:03 djugei

From our contrib docs

Already have an idea? It might be good to first create an issue to propose it so we can make sure we are aligned and lower the risk of having to re-work some of it and the discouragement that goes along with that.

Going to mark this as a draft for now.

epage avatar Mar 31 '25 17:03 epage

Its not really a draft though, it works as is. I am already using it. Main change i would do is document it better though i do not know where i would best put that info.

djugei avatar Mar 31 '25 17:03 djugei

We do not have agreement that this solution is in the direction we want to go and so this is only a proposal for demonstrating a direction, regardless of how useful it is to you in the moment. Having this be a draft (or closed) clarifies how this is being treated and pushes discussion on solutions to a single place, the issue.

epage avatar Mar 31 '25 17:03 epage

oh, ok so a draft from the project side, i guess that makes sense. i usually see the draft mode used from the contributor side to indicate work-in-progress status.

djugei avatar Mar 31 '25 17:03 djugei

Yes, but considering we don't yet know what directions this may change, we can't say this is ready for review.

epage avatar Mar 31 '25 18:03 epage

i kinda want to slightly alter the log format, right now syslog contains

date host processname[process_id] processname::module:: message

i would like date host processname[process_id] ::module:: message better to reduce redundancy.

i am unsure what the easiest way to get the top level binary name is though. then it would be a simple .strip_prefix(name).

djugei avatar Apr 09 '25 13:04 djugei

should be merged after this: https://github.com/rust-cli/env_logger/pull/363

djugei avatar Apr 19 '25 13:04 djugei