diff options
| author | DmitriyEshenko <dmitriy.eshenko@vyos.io> | 2020-04-10 18:16:15 +0000 |
|---|---|---|
| committer | DmitriyEshenko <dmitriy.eshenko@vyos.io> | 2020-04-10 18:16:15 +0000 |
| commit | 5e61b09a5d0ad4d4cfd25eafcc95f0d2b1a927b9 (patch) | |
| tree | b9420932d6f7ce46d001814f6a553b515eb1922d | |
| parent | 927c98f23a6317e2e3565032023d68b753d71037 (diff) | |
| download | vyos-1x-5e61b09a5d0ad4d4cfd25eafcc95f0d2b1a927b9.tar.gz vyos-1x-5e61b09a5d0ad4d4cfd25eafcc95f0d2b1a927b9.zip | |
pptp: l2tp: T2262: Fix call to function run()
| -rw-r--r-- | op-mode-definitions/reset-vpn.xml | 4 | ||||
| -rwxr-xr-x | src/op_mode/reset_vpn.py | 23 |
2 files changed, 13 insertions, 14 deletions
diff --git a/op-mode-definitions/reset-vpn.xml b/op-mode-definitions/reset-vpn.xml index c0b0ddeb1..a081ea488 100644 --- a/op-mode-definitions/reset-vpn.xml +++ b/op-mode-definitions/reset-vpn.xml @@ -62,13 +62,13 @@ <properties> <help>Terminate all user's current remote access VPN session(s) with L2TP protocol</help> </properties> - <command>sudo ${vyos_op_scripts_dir}/reset_vpn.py --username="all_users" --protocol="l2tp"</command> + <command>sudo ${vyos_op_scripts_dir}/reset_vpn.py --username="$5" --protocol="l2tp"</command> </leafNode> <leafNode name="pptp"> <properties> <help>Terminate all user's current remote access VPN session(s) with PPTP protocol</help> </properties> - <command>sudo ${vyos_op_scripts_dir}/reset_vpn.py --username="all_users" --protocol="pptp"</command> + <command>sudo ${vyos_op_scripts_dir}/reset_vpn.py --username="$5" --protocol="pptp"</command> </leafNode> </children> </node> diff --git a/src/op_mode/reset_vpn.py b/src/op_mode/reset_vpn.py index 8962df212..15908ee77 100755 --- a/src/op_mode/reset_vpn.py +++ b/src/op_mode/reset_vpn.py @@ -30,41 +30,40 @@ def terminate_sessions(username='', interface='', protocol=''): if username == "all_users": if protocol == "pptp": pptp_cmd = pptp_base.format('all','') - run(pptp_cmd, stdout=DEVNULL, stderr=DEVNULL) + run(pptp_cmd) return elif protocol == "l2tp": l2tp_cmd = l2tp_base.format('all', '') - run(l2tp_cmd, stdout=DEVNULL, stderr=DEVNULL) + run(l2tp_cmd) return else: pptp_cmd = pptp_base.format('all', '') - run(pptp_cmd, stdout=DEVNULL, stderr=DEVNULL) + run(pptp_cmd) l2tp_cmd = l2tp_base.format('all', '') - run(l2tp_cmd, stdout=DEVNULL, stderr=DEVNULL) + run(l2tp_cmd) return if protocol == "pptp": pptp_cmd = pptp_base.format('username', username) - run(pptp_cmd, stdout=DEVNULL, stderr=DEVNULL) + run(pptp_cmd) return elif protocol == "l2tp": l2tp_cmd = l2tp_base.format('username', username) - run(l2tp_cmd, stdout=DEVNULL, stderr=DEVNULL) + run(l2tp_cmd) return else: pptp_cmd = pptp_base.format('username', username) - run(pptp_cmd, stdout=DEVNULL, stderr=DEVNULL) - l2tp_cmd.append("terminate username {0}".format(username)) - run(l2tp_cmd, stdout=DEVNULL, stderr=DEVNULL) + run(pptp_cmd) + l2tp_cmd = l2tp_base.format('username', username) + run(l2tp_cmd) return # rewrite `terminate by interface` if pptp will have pptp%d interface naming if interface: pptp_cmd = pptp_base.format('if', interface) - run(pptp_cmd, stdout=DEVNULL, stderr=DEVNULL) + run(pptp_cmd) l2tp_cmd = l2tp_base.format('if', interface) - run(l2tp_cmd, stdout=DEVNULL, stderr=DEVNULL) - + run(l2tp_cmd) def main(): #parese args |