go-graphite / carbonapi

Implementation of graphite API (graphite-web) in golang
Other
309 stars 140 forks source link

[BUG] time unit parsing issue in render API #772

Closed je1lylee closed 1 year ago

je1lylee commented 1 year ago

Describe the bug

I have a graphite alert tools called seyren, seyren make four images in different time range like this: image

If seyren get images from carbonapi, this four images looks same. I checked graphte-web from here, they process time unit into lowercase to prevent this issue..

And seyren's reuqest like this:

?&width=1200&height=350&from=-15Minutes

CarbonAPI Version My carbonapi was build base this coomit.

commit d2573f9e00e4420906909d7a8c28ce57c4db2aff
Merge: c4c2d75 7106a53
Author: Carrie Edwards <edwrdscarrie@gmail.com>
Date:   Wed May 31 14:57:33 2023 +0000

    Merge pull request #769 from go-graphite/cedwards/upper-lower-func-name

    Fix toUpperCase and toLowerCase to process queries that use correct function names

If you guys think we should support this case, i can make a PR.

Thanks a lot.