From: Florian Eckert <fe@dev.tdt.de>
Date: Fri, 3 May 2024 10:43:06 +0000 (+0200)
Subject: luci-proto-modemmanger: fix auth handling
X-Git-Url: http://git.lede-project.org./?a=commitdiff_plain;h=7546249a27ab1f9d6e0035f9f600a3f2c1c9b772;p=project%2Fluci.git

luci-proto-modemmanger: fix auth handling

This has never worked because the option 'auth' is not known to the
modemmanger proto handler. The correct uci name is 'allowedauth' and is
also a uci list option, so that several options can be selected.

This commit fixes this bug.

Signed-off-by: Florian Eckert <fe@dev.tdt.de>
---

diff --git a/protocols/luci-proto-modemmanager/htdocs/luci-static/resources/protocol/modemmanager.js b/protocols/luci-proto-modemmanager/htdocs/luci-static/resources/protocol/modemmanager.js
index b13dd310c7..fa3c001025 100644
--- a/protocols/luci-proto-modemmanager/htdocs/luci-static/resources/protocol/modemmanager.js
+++ b/protocols/luci-proto-modemmanager/htdocs/luci-static/resources/protocol/modemmanager.js
@@ -76,11 +76,13 @@ return network.registerProtocol('modemmanager', {
 		o = s.taboption('general', form.Value, 'pincode', _('PIN'));
 		o.datatype = 'and(uinteger,minlength(4),maxlength(8))';
 
-		o = s.taboption('general', form.ListValue, 'auth', _('Authentication Type'));
-		o.value('both', _('PAP/CHAP (both)'));
+		o = s.taboption('general', form.DynamicList, 'allowedauth', _('Authentication Type'));
 		o.value('pap', 'PAP');
 		o.value('chap', 'CHAP');
-		o.value('none', _('None'));
+		o.value('mschap', 'MSCHAP');
+		o.value('mschapv2', 'MSCHAPv2');
+		o.value('eap', 'EAP');
+		o.value('', _('None'));
 		o.default = 'none';
 
 		o = s.taboption('general', form.ListValue, 'allowedmode', _('Allowed network technology'),