Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

parsing multiple field values in EnvFilter is broken #2935

Open
Voker57 opened this issue Apr 13, 2024 · 0 comments · May be fixed by #2959
Open

parsing multiple field values in EnvFilter is broken #2935

Voker57 opened this issue Apr 13, 2024 · 0 comments · May be fixed by #2959

Comments

@Voker57
Copy link

Voker57 commented Apr 13, 2024

Bug Report

Version

908cc43

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: Voker57@3ca76bf

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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
1 participant
@Voker57