diff options
author | Jan Hoffmann <jan@3e8.eu> | 2023-11-18 09:52:45 +0100 |
---|---|---|
committer | Jan Hoffmann <jan@3e8.eu> | 2023-11-18 09:52:45 +0100 |
commit | 3ac4f567dfdfd6ee989ceffb025241e605790b6d (patch) | |
tree | e338880f0df823f6cb2dd345f18abb284a4b6c8d /applications/luci-app-vnstat2/htdocs/luci-static/resources | |
parent | 63d5e239633371ae19b61038c1c7e8c11c2fc75f (diff) |
luci-app-vnstat2: use more efficient way to get database interfaces
The currently used command "vnstat --json f 1" can be very slow when the
database contains many entries. This can result in an XHR timeout error
on the configuration page.
Use the "--dbiflist" option instead, which was added in vnStat 2.7.
Fixes: #6702
Signed-off-by: Jan Hoffmann <jan@3e8.eu>
Diffstat (limited to 'applications/luci-app-vnstat2/htdocs/luci-static/resources')
-rw-r--r-- | applications/luci-app-vnstat2/htdocs/luci-static/resources/view/vnstat2/config.js | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/applications/luci-app-vnstat2/htdocs/luci-static/resources/view/vnstat2/config.js b/applications/luci-app-vnstat2/htdocs/luci-static/resources/view/vnstat2/config.js index 10cd85641c..4d4bfd0dbe 100644 --- a/applications/luci-app-vnstat2/htdocs/luci-static/resources/view/vnstat2/config.js +++ b/applications/luci-app-vnstat2/htdocs/luci-static/resources/view/vnstat2/config.js @@ -58,13 +58,10 @@ return view.extend({ o = s.option(form.DummyValue, '_database'); o.load = function(section_id) { - return fs.exec('/usr/bin/vnstat', ['--json', 'f', '1']).then(L.bind(function(result) { + return fs.exec('/usr/bin/vnstat', ['--dbiflist', '1']).then(L.bind(function(result) { var databaseInterfaces = []; if (result.code == 0) { - var vnstatData = JSON.parse(result.stdout); - for (var i = 0; i < vnstatData.interfaces.length; i++) { - databaseInterfaces.push(vnstatData.interfaces[i].name); - } + databaseInterfaces = result.stdout.trim().split('\n'); } var configInterfaces = uci.get_first('vnstat', 'vnstat', 'interface') || []; |