summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorChristian Poessinger <christian@poessinger.com>2019-09-06 18:02:40 +0200
committerChristian Poessinger <christian@poessinger.com>2019-09-06 18:06:15 +0200
commit2e3b701f9045c4a557c99ee311c11d4a6f4f7b78 (patch)
treec11bca2f4cf5befe524115c40094dfaa737d1a5a
parent2e48db607e3f228cf6f40b53478966e5fbe00dbc (diff)
downloadvyos-1x-2e3b701f9045c4a557c99ee311c11d4a6f4f7b78.tar.gz
vyos-1x-2e3b701f9045c4a557c99ee311c11d4a6f4f7b78.zip
openvpn: T1548: cleanup import statements
-rwxr-xr-xsrc/conf_mode/interface-openvpn.py28
1 files changed, 14 insertions, 14 deletions
diff --git a/src/conf_mode/interface-openvpn.py b/src/conf_mode/interface-openvpn.py
index 5f5cbd67d..fe08cd51e 100755
--- a/src/conf_mode/interface-openvpn.py
+++ b/src/conf_mode/interface-openvpn.py
@@ -18,15 +18,15 @@
import os
import re
-import pwd
-import grp
import sys
import stat
-import copy
import jinja2
-import psutil
-from ipaddress import ip_address,ip_network,IPv4Interface
+from copy import deepcopy
+from grp import getgrnam
+from ipaddress import ip_address,ip_network,IPv4Interface
+from psutil import pid_exists
+from pwd import getpwnam
from signal import SIGUSR1
from subprocess import Popen, PIPE
@@ -302,8 +302,8 @@ def openvpn_mkdir(directory):
# fix permissions - corresponds to mode 755
os.chmod(directory, stat.S_IRWXU|stat.S_IRGRP|stat.S_IXGRP|stat.S_IROTH|stat.S_IXOTH)
- uid = pwd.getpwnam(user).pw_uid
- gid = grp.getgrnam(group).gr_gid
+ uid = getpwnam(user).pw_uid
+ gid = getgrnam(group).gr_gid
os.chown(directory, uid, gid)
def fixup_permission(filename, permission=stat.S_IRUSR):
@@ -315,8 +315,8 @@ def fixup_permission(filename, permission=stat.S_IRUSR):
os.chmod(filename, permission)
# make file owned by root / vyattacfg
- uid = pwd.getpwnam('root').pw_uid
- gid = grp.getgrnam('vyattacfg').gr_gid
+ uid = getpwnam('root').pw_uid
+ gid = getgrnam('vyattacfg').gr_gid
os.chown(filename, uid, gid)
def checkCertHeader(header, filename):
@@ -335,7 +335,7 @@ def checkCertHeader(header, filename):
return False
def get_config():
- openvpn = copy.deepcopy(default_config_data)
+ openvpn = deepcopy(default_config_data)
conf = Config()
# determine tagNode instance
@@ -793,8 +793,8 @@ def generate(openvpn):
fixup_permission(auth_file)
# get numeric uid/gid
- uid = pwd.getpwnam(user).pw_uid
- gid = grp.getgrnam(group).gr_gid
+ uid = getpwnam(user).pw_uid
+ gid = getgrnam(group).gr_gid
# Generate client specific configuration
for client in openvpn['client']:
@@ -834,7 +834,7 @@ def apply(openvpn):
# we only need to stop the demon if it's running
# daemon could have died or killed by someone
- if psutil.pid_exists(pid):
+ if pid_exists(pid):
cmd = 'start-stop-daemon --stop --quiet'
cmd += ' --pidfile ' + pidfile
subprocess_cmd(cmd)
@@ -857,7 +857,7 @@ def apply(openvpn):
return None
# Send SIGUSR1 to the process instead of creating a new process
- if psutil.pid_exists(pid):
+ if pid_exists(pid):
os.kill(pid, SIGUSR1)
return None