search

go-graphite / carbonzipper

proxy to transparently merge graphite carbon backends
Other
104 stars 29 forks source link

Zipper.Render() wrong request param to carbonserver #40

Closed StephenPCG closed 7 years ago

StephenPCG commented 7 years ago

https://github.com/go-graphite/carbonzipper/blob/master/zipper/zipper.go#L530-L533

        "target": []string{""},
        ...
        "until":  []string{strconv.Itoa(int(from))},

Should be:

        "target": []string{target},
        ...
        "until":  []string{strconv.Itoa(int(until))},
Civil commented 7 years ago

Thanks for mentioning that.

We are currently doing much of refactoring on master, so it can contain bugs (the only thing that's guaranteed for now is that master compiles).

So until we tag next release, I'd suggest you to stay away from master if you want to have stable software stack.