grafana-zabbix icon indicating copy to clipboard operation
grafana-zabbix copied to clipboard

Published 20 hours ago • verified publisher icon grafana

Switch to regexp2 for parseFilter()

Open thetuxkeeper opened this issue 3 years ago • 1 comments

I changed the returned regexp of parseFilter() to regexp2. I kept all other uses regexp since there's no need for a switch and regexp2 seems to be unnecessary complex to use for something like FindStringSubmatch(). I haven't run any tests on a real grafana instance, I'll try that later. But I extended the tests and they passed.

If there's anything I could/should change, just let me know! :)

This PR should fix #1264

thetuxkeeper avatar Jul 07 '22 13:07 thetuxkeeper