diff options
| author | Kevin Locke <kevin@kevinlocke.name> | 2018-11-14 16:55:24 -0700 |
|---|---|---|
| committer | Kevin Locke <kevin@kevinlocke.name> | 2018-11-14 16:55:24 -0700 |
| commit | 1bbc419e7aa4f3d9e9a269a87a8f5c191440f78b (patch) | |
| tree | ac34b2666f9a62aa344dcee5c9fdfb1f1d579212 /build | |
| parent | 6bc04b6afb1332473e8283fa2f9517db0816cb23 (diff) | |
luci-app-statistics: quote : in filenames for DEF
: is used to delimit fields in DEF: rrd instructions, so when it appears
in a filename it must be escaped using \.[1] This commit adds the
escaping.
I discovered the issue after configuring collectd-mod-ping to monitor an
IPv6 host (2001:19f0:5:727:5b56:205d:ff55:2208). Accessing
https://192.168.0.1/cgi-bin/luci/admin/statistics/graph/ping would cause
the following messages to be logged:
Wed Nov 14 23:04:33 2018 daemon.err uhttpd[30261]: ERROR: can't parse DEF '2ping_avg_raw=/tmp/rrd/openwrthost/ping/ping-2001:19f0:5:727:5b56:205d:ff55:2208.rrd:value:AVERAGE' -2
Wed Nov 14 23:04:33 2018 daemon.err uhttpd[30261]: ERROR: can't parse DEF '2ping_droprate_avg_raw=/tmp/rrd/openwrthost/ping/ping_droprate-2001:19f0:5:727:5b56:205d:ff55:2208.rrd:value:AVERAGE' -2
and the graphs would not display. After applying this commit, the
graphs display correctly and no messages are logged.
1. https://oss.oetiker.ch/rrdtool/doc/rrdgraph_data.en.html#IDEF
Signed-off-by: Kevin Locke <kevin@kevinlocke.name>
Diffstat (limited to 'build')
0 files changed, 0 insertions, 0 deletions