Feature #9911
closedShow confirmation box before disconnecting PPPoE
100%
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.
Updated by Jim Pingle about 5 years ago
- 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.
Updated by Jim Pingle about 5 years ago
- Status changed from New to Feedback
- % Done changed from 0 to 100
Applied in changeset b8b0c2a320166a3b5732354d35edad47d0f05a04.
Updated by Anonymous about 5 years ago
Unsure if it's appropriate to say "Thanks" in the bugtracker, but thanks!! Especially for such a prompt patch. Appreciate it.
Updated by Jim Pingle about 5 years ago
You're welcome! Did you have a chance to test the patch? You should be able to apply commit 4193cc185ef55e2260dae4ff21528f74ad50cde0 against RELENG_2_4_4 using the System Patches package: https://docs.netgate.com/pfsense/en/latest/development/system-patches.html
If you try it and it works, leave feedback here so we can close the issue out.
Updated by Anonymous about 5 years ago
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.
Updated by Ronald Schellberg about 5 years ago
I can confirm this patch works. Nice red button and it requests confirmation of the selection to disconnect.
Updated by Jim Pingle about 5 years ago
- Status changed from Feedback to Resolved
Great, thanks for testing!
Updated by Jim Pingle about 5 years ago
- Target version changed from 2.5.0 to 2.4.5
Updated by Jim Pingle almost 5 years ago
- Status changed from Resolved to Feedback
Needs checked and/or tested again on 2.4.5 snapshots
Updated by Jim Pingle almost 5 years ago
- 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.