go-graphite / carbonapi

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

Skip all whitespace during expr parsing #786

Closed leizor closed 1 year ago

leizor commented 1 year ago

This PR expands the whitespace trimming done during expression parsing to include all unicode whitespace and not just " ".

It also makes a minor behavioral change due to trimming whitespace a bit more aggressively now; when the target is only whitespace, we return ErrMissingExpr instead of ErrMissingArgument.