fixed issue with regexp matching.

[luqasz]

this patch will fix issue with regexp matching.
it sometimes happened to wrongly parse all data.

please see:
https://collectd.org/wiki/index.php/Naming_schema
for more information.

i do not know what is "category" for so i left it empty not to break anything. hopefully.
"varnish" is also something that i do not know what is for.

[pommi]

Thanks for the pull request, but I can't include it. I don't really get "it sometimes happened to wrongly parse all data". Please give some examples so the regex can be fixed instead of creating 3 functions that do the same.

[luqasz]

have you read ?
https://collectd.org/wiki/index.php/Naming_schema
you do not need regexp to pull data and parse it.

regarding "creating 3 functions that do the same"
no they do not do the same.
they are more readable then regexp

"it sometimes happened to wrongly parse all data"
yes it does happen. unfortunatelly i do not have a reproductive example.

[pommi]

I'm fully aware of that page. CGP is completely based on that page, as is the regex. Except that the varnish plugin didn't fit it (explained here: e3f1cc2)

If you find a reproducible example, please report it and I will happily fix the regex.

[luqasz]

ok i got a reproducable example.
create directory and rrd file as follows:
hostname/snmp/if_octets-.rrd

this will give you error messages in apache log file:
sh: 1: cannot open some-name: No such file
this is becouse there are no shellescapes and further no urlencode/decode
this fixes it.
https://github.com/uqasz/CGP/commit/77a3477d9d25996266c9bc30a4fc1aa090864385

[pommi]

Should be fixed in #87