Project

General

Profile

Actions

Feature #9911

closed

Show confirmation box before disconnecting PPPoE

Added by Anonymous about 2 years ago. Updated almost 2 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
Web Interface
Target version:
Start date:
11/18/2019
Due date:
% Done:

100%

Estimated time:
Plus Target Version:
Release Notes:

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.

Actions #1

Updated by Jim Pingle about 2 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.

Actions #2

Updated by Jim Pingle about 2 years ago

  • Status changed from New to Feedback
  • % Done changed from 0 to 100
Actions #3

Updated by Anonymous about 2 years ago

Unsure if it's appropriate to say "Thanks" in the bugtracker, but thanks!! Especially for such a prompt patch. Appreciate it.

Actions #4

Updated by Jim Pingle about 2 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.

Actions #5

Updated by Anonymous about 2 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.

Actions #6

Updated by Ronald Schellberg about 2 years ago

I can confirm this patch works. Nice red button and it requests confirmation of the selection to disconnect.

Actions #7

Updated by Jim Pingle about 2 years ago

  • Status changed from Feedback to Resolved

Great, thanks for testing!

Actions #8

Updated by Jim Pingle almost 2 years ago

  • Target version changed from 2.5.0 to 2.4.5
Actions #9

Updated by Jim Pingle almost 2 years ago

  • Status changed from Resolved to Feedback

Needs checked and/or tested again on 2.4.5 snapshots

Actions #10

Updated by Jim Pingle almost 2 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.

Actions

Also available in: Atom PDF