IPv6 doesn't work because the UdpSocket is always bound to IPv4. send_to then always fails, but the error is silently ignored.

Took me quite a while to find this ...

This could be fixed by checking if the address is IPv6 or IPv4 and then binding to 0.0.0.0:0 for IPv4 and to [::]:0 for IPv6.

Hey there,

Thank you for writing this, awesome project!

Curious, should there be support for tagging metrics? I could imagine something like this:

client.incr("some.metric", vec!["service:api"])

or maybe even better:

let client = Client::new("127.0.0.1:8125", "project", vec!["service:api"]).unwrap()

and this would add the tag service with value api to all metrics.

Thanks!

Edit: I think I was confused; tags seem to be a Datadog extension of StatsD, and it's not part of the StatsD spec.

According to https://github.com/statsd/statsd/blob/master/docs/metric_types.md.
It needs |@{rate} suffix for sampled metrics.
I have created pull request containing fix of this.

Hi, I am pretty new to rust. Using this lib was super simple, thanks for the work!

I don't know how to do mutable calls in the client.time closure callable because it is set as Fn and not FnOnce or FnMut.

I get this error: cannot assign to data in a captured outer variable in an Fn closure

Here is a little example of what I am trying to do. Any guidance appreciated:

extern crate statsd;
use statsd::Client;

#[derive(Debug)]
pub struct Foo {
    num: u64,
}

fn main() {
    let mut foo_num = Foo{num: 10};
    println!("before: {:?}", foo_num);
    let client = Client::new("127.0.0.1:8125", "foo").unwrap();
    client.time("timethis", || {
        // Assume this is an expensive operation I want to time that takes ownership of a mutable ref
        foo_num.num+=1;
        println!("foo: {:?}", foo_num);
    });
}

The example from the doc says:

use statsd::client::Client;

let client = Client::new("127.0.0.1:8125", "myapp.");
client.incr("some.metric.completed");

This will generate a metric named myapp..some.metric.completed. The extra dot will cause issues since Graphite treats this the same as myapp.some.metric.completed and stores data in the same file, but statsd thinks they are different.

I'm not 100% sure but I think this isn't the intended behavior.

I'll make a pull request that fixes this.

Client.time is declared as

fn time<F>(&mut self, metric: &str, callable: F) where F: Fn()

So it can't be used to run a function that returns a value. It'd be better to be:

fn time<F, R>(&mut self, metric: &str, callable: F) -> R where F: Fn() -> R