Feature #9911
closed
Show confirmation box before disconnecting PPPoE
Added by Anonymous over 6 years ago.
Updated over 6 years ago.
Description
The Status->Interfaces page (status_interfaces.php) is very useful for showing Interface details.
On systems that utilise PPPoE, a Disconnect box is shown next to the PPPoE up status.
This button, if accidentally pressed, disconnects the PPPoE session instantly. No confirmation is shown.
To my mind, the Status page is just that, showing the status of things. It shouldn't be a section that can act on the state of the system. Therefore I think this button should at least show a "Disconnect PPPoE?" or similar dialog box, before performing the action, in the same way as clicking the Trash Can on the states page (diag_dump_states.php) gives you a "Are you sure you wish to delete this state?" message.
We discovered this today when someone was working on a remote site and, not even intentionally, clicked the button while they were going up to close the browser tab. The result was the site was offline until we were able to get someone there and click the button again to connect it.
- Category set to Web Interface
- Assignee set to Jim Pingle
- Target version set to 2.5.0
This should be as easy as changing the button from a warning class to a danger class, which automatically gets a JS confirmation popup when used on a button. I'll have a look at it.
- Status changed from New to Feedback
- % Done changed from 0 to 100
Applied in changeset commit:b8b0c2a320166a3b5732354d35edad47d0f05a04.
Unsure if it's appropriate to say "Thanks" in the bugtracker, but thanks!! Especially for such a prompt patch. Appreciate it.
Hi Jim.
I've applied the patch and I'm happy to confirm that yes, it works perfectly!
I like the fact it's now RED and clearly shows it's going to disconnect the WAN interface.
Clicking on it brings up a javascript confirmation box, and does not disconnect it straight away.
I haven't clicked "OK" to confirm it actually disconnects, but I can confirm that "Cancel" doesn't!
Thanks again for the prompt fix.
I can confirm this patch works. Nice red button and it requests confirmation of the selection to disconnect.
- Status changed from Feedback to Resolved
Great, thanks for testing!
- Target version changed from 2.5.0 to 2.4.5
- Status changed from Resolved to Feedback
Needs checked and/or tested again on 2.4.5 snapshots
- Status changed from Feedback to Resolved
Looks good on 2.4.5.a.20191217.0637
Buttons are red and trigger the JS confirmation dialog. Canceling the dialog results in no action being taken by the firewall.
Also available in: Atom
PDF