luci-proto-ppp: pppoe.so has no MTU option; remove it
authorPaul Donald <newtwen+github@gmail.com>
Tue, 16 Apr 2024 15:41:30 +0000 (17:41 +0200)
committerPaul Donald <newtwen+github@gmail.com>
Wed, 17 Apr 2024 00:51:15 +0000 (02:51 +0200)
Retain comment so nobody is tempted to resurrect it.

Signed-off-by: Paul Donald <newtwen+github@gmail.com>
(cherry picked from commit 995d3a1801cf3b850e8c7c35937fbdd0526ff53e)

protocols/luci-proto-ppp/htdocs/luci-static/resources/protocol/pppoe.js

index 057e17fe81b1906c1e0c2a3d8b5cc460fbde03b0..b025a3eeb2f67f27e963c1c3b32a8aa74caa811b 100644 (file)
@@ -93,8 +93,6 @@ return network.registerProtocol('pppoe', {
                o.placeholder = '0';
                o.datatype    = 'uinteger';
 
-               o = s.taboption('advanced', form.Value, 'mtu', _('Override MTU'));
-               o.placeholder = dev ? (dev.getMTU() || '1500') : '1500';
-               o.datatype    = 'max(9200)';
+               /* pppoe.so has no MTU option - only underlying device MTU is possible */
        }
 });