v5 causing malformed tag value using "true" / "false"

[RobGraham]

After switching to v5, some of our charts in NewRelic stopped reporting because one of the tags we use to report a "true" or "false" value started having other erroneous data concatenated. This could be NewRelic but other than the version change of statsd, our implementation is unchanged. Switching back to v4, our metrics started reporting correctly.

I spent some time debugging into the library and saw that our isBot tag is correctly showing a true or false value: "isBot:false" - however, there seems to be a "c|" separator which is causing mischief with this tag in NewRelic.

NewRelic query

Debugging locally:

Aggregator Flush

statsdBuffer Write

[carlosroman]

This is expected as v5 uses the new DogStatsD protocol v1.2. The container tag should not be appended if you disable Origin detection when creating the client (use option WithoutOriginDetection).