luci-app-banip: handle load errors of the countries file
authorDirk Brenken <dev@brenken.org>
Fri, 26 Apr 2024 11:42:54 +0000 (13:42 +0200)
committerDirk Brenken <dev@brenken.org>
Fri, 26 Apr 2024 19:32:34 +0000 (21:32 +0200)
* properly handle/skip possible read errors in try/catch blocks

Signed-off-by: Dirk Brenken <dev@brenken.org>
(cherry picked from commit f8eb80bac75df3b69ec0713524c8bb1169d7d3b9)

applications/luci-app-banip/htdocs/luci-static/resources/view/banip/overview.js

index c8c571f7353db848d83e2deca2e7c1b2a83f274e..bb5804a90e74631a34c218ec7636feb230896de5 100644 (file)
@@ -643,10 +643,15 @@ return view.extend({
 
                        o = s.taboption('feeds', form.MultiValue, 'ban_country', _('Countries (RIR)'));
                        for (let i = 0; i < countries.length; i++) {
-                               ccode = countries[i].match(/^(\w+)\t/)[1].trim();
-                               rir = countries[i].match(/^\w+\t(\w+)\t/)[1].trim();
-                               country = countries[i].match(/^\w+\t\w+\t(.*$)/)[1].trim();
-                               o.value(ccode, country + ' (' + rir + ')');
+                               try {
+                                       ccode = countries[i].match(/^(\w+)\t/)[1].trim();
+                                       rir = countries[i].match(/^\w+\t(\w+)\t/)[1].trim();
+                                       country = countries[i].match(/^\w+\t\w+\t(.*$)/)[1].trim();
+                                       o.value(ccode, country + ' (' + rir + ')');
+                               } catch (e) {
+                                       countries[i] = "";
+                                       ui.addNotification(null, E('p', _('Unable to parse the countries file: %s').format(e.message)), 'error');
+                               }
                        }
                        o.optional = true;
                        o.rmempty = true;
@@ -673,11 +678,15 @@ return view.extend({
                o = s.taboption('feeds', form.DynamicList, 'ban_allowurl', _('Allowlist Feed URLs'));
                if (countries) {
                        for (let i = 0; i < countries.length; i++) {
-                               ccode = countries[i].match(/^(\w+)\t/)[1].trim();
-                               rir = countries[i].match(/^\w+\t(\w+)\t/)[1].trim();
-                               country = countries[i].match(/^\w+\t\w+\t(.*$)/)[1].trim();
-                               o.value('https://www.ipdeny.com/ipblocks/data/aggregated/' + ccode + '-aggregated.zone', country + ' IPv4 (' + rir + ')');
-                               o.value('https://www.ipdeny.com/ipv6/ipaddresses/aggregated/' + ccode + '-aggregated.zone', country + ' IPv6 (' + rir + ')');
+                               try {
+                                       ccode = countries[i].match(/^(\w+)\t/)[1].trim();
+                                       rir = countries[i].match(/^\w+\t(\w+)\t/)[1].trim();
+                                       country = countries[i].match(/^\w+\t\w+\t(.*$)/)[1].trim();
+                                       o.value('https://www.ipdeny.com/ipblocks/data/aggregated/' + ccode + '-aggregated.zone', country + ' IPv4 (' + rir + ')');
+                                       o.value('https://www.ipdeny.com/ipv6/ipaddresses/aggregated/' + ccode + '-aggregated.zone', country + ' IPv6 (' + rir + ')');
+                               } catch (e) {
+                                       countries[i] = "";
+                               }
                        }
                }
                o.optional = true;