parsing multiple field values in EnvFilter is broken - Githubissues

tokio-rs / tracing

Application level tracing for Rust.

https://tracing.rs

MIT License

5.15k stars 673 forks source link

parsing multiple field values in EnvFilter is broken #2935

Open Voker57 opened 2 months ago

Voker57 commented 2 months ago

Bug Report

Version

Platform

Linux 6.7.9-amd64 #1 SMP PREEMPT_DYNAMIC Debian 6.7.9-2 (2024-03-13) x86_64 GNU/Linux

Crates

tracing-subscriber

Description

EnvFilter fails to parse multiple field values. This is covered by tests, but EnvFilter::new() only outputs errors to stderr thus tests pass. Test fix: https://github.com/Voker57/tracing/commit/3ca76bf18a6acb97c5e3d942e308dca6982c530b

Output of EnvFilter::new() with input from callsite_enabled_includes_span_directive_multiple_fields:

ignoring `app[mySpan{field="value"`: invalid filter directive
ignoring `field2=2}]=debug`: invalid filter directive