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

[dogstatsd] Service checks: message field parsing confused by m: #2373

Closed
olivielpeau opened this issue Mar 21, 2016 · 0 comments
Closed

[dogstatsd] Service checks: message field parsing confused by m: #2373

olivielpeau opened this issue Mar 21, 2016 · 0 comments
Assignees
@olivielpeau
Labels
bugfix
Milestone
5.8.0

Comments

@olivielpeau
Copy link
Member

See 1ca4a67

The test fails because m: is regarded as the start of a message field when it shouldn't be (it's just part of the tag).

See also

dd-agent/aggregator.py

Line 557 in b09211d

message_delimiter = '|m:' if '|m:' in metadata else 'm:'
for the line of code that's probably responsible
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@olivielpeau olivielpeau

Labels
bugfix
Projects
None yet
Milestone
5.8.0
Development

No branches or pull requests
2 participants
@remh @olivielpeau