Bug #6732
closed
interfaces_ppps_edit.php: L2TP and PPTP WAN-type interface editing has broken input validation
100%
Description
On interfaces_ppps_edit.php when an L2TP or PPTP interface is created or edited, it cannot be saved.
Both types have an expanding list of "IP/Gateway" inputs for each selected interface, but input validation is not checking the arrays properly and it is rejecting any input. When the input validation is removed, the values are not stored in the config either.
Going the other way seems to work, starting at interfaces.php and then making an L2TP interface there, it works. If you edit that working entry at interfaces_ppps_edit.php it shows the values entered on the other page, but on save they are lost, as above.
Updated by Anonymous over 8 years ago
- Status changed from New to Feedback
- Assignee changed from Anonymous to Jim Pingle
Element names and validation code fixed to resolve issue.
localip, submnt, gateway, mtu, mru & mrru tested
Updated by Anonymous over 8 years ago
- % Done changed from 0 to 100
Applied in changeset e73770057872aa1f78dfff885f89916e446e2e4c.
Updated by Jim Pingle almost 8 years ago
- Target version changed from 2.4.0 to 2.3.3
Updated by Thomas Rieschl almost 8 years ago
It seems that it only works partially.
I just tried to paly around with the interfaces_ppps_edit.php-Settings on version 2.4.0-BETA-amd64-20170309. The values seem to get saved but are not displayed in the input fields when opening the page.
It's the same behavior in version 2.3.3-p1 btw.
Updated by Phillip Davis almost 8 years ago
Can you give some detail of what you are entering. I can't (easily) reproduce it, so maybe there is some unusual/specific set of inputs that are a problem.