From: Jo-Philipp Wich <jo@mein.io>
Date: Mon, 13 May 2024 10:34:15 +0000 (+0200)
Subject: Revert "luci-proto-ppp: pppoe.so has no MTU option; remove it"
X-Git-Url: http://git.lede-project.org./?a=commitdiff_plain;h=b57c6ff29eb625913b91e0f6ebb754a5454acb9f;p=project%2Fluci.git

Revert "luci-proto-ppp: pppoe.so has no MTU option; remove it"

This reverts commit 995d3a1801cf3b850e8c7c35937fbdd0526ff53e.

The `mtu` option controls the pppd mru and mtu arguments, it is not
specific to pppoe.so.

Signed-off-by: Jo-Philipp Wich <jo@mein.io>
---

diff --git a/protocols/luci-proto-ppp/htdocs/luci-static/resources/protocol/pppoe.js b/protocols/luci-proto-ppp/htdocs/luci-static/resources/protocol/pppoe.js
index b025a3eeb2..057e17fe81 100644
--- a/protocols/luci-proto-ppp/htdocs/luci-static/resources/protocol/pppoe.js
+++ b/protocols/luci-proto-ppp/htdocs/luci-static/resources/protocol/pppoe.js
@@ -93,6 +93,8 @@ return network.registerProtocol('pppoe', {
 		o.placeholder = '0';
 		o.datatype    = 'uinteger';
 
-		/* pppoe.so has no MTU option - only underlying device MTU is possible */
+		o = s.taboption('advanced', form.Value, 'mtu', _('Override MTU'));
+		o.placeholder = dev ? (dev.getMTU() || '1500') : '1500';
+		o.datatype    = 'max(9200)';
 	}
 });