starred/certbot-certbot
1
0
Fork 0
mirror of https://github.com/certbot/certbot.git synced 2026-07-30 18:04:31 +02:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #375 from kuba/apache-no-sudo

Browse Source 
Don't use sudo in apache plugin
This commit is contained in:
James Kasten
2015-05-02 09:17:43 -07:00
parent 828e09543b 7a4d37e320
commit 93cdad120d
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
letsencrypt/client/plugins/apache/configurator.py
+2 -2
View File
@@ -908,7 +908,7 @@ class ApacheConfigurator(augeas_configurator.AugeasConfigurator):
""" """
try: try:
proc = subprocess.Popen( proc = subprocess.Popen(
["sudo", self.config.apache_ctl, "configtest"], # TODO: sudo? [self.config.apache_ctl, "configtest"],
stdout=subprocess.PIPE, stdout=subprocess.PIPE,
stderr=subprocess.PIPE) stderr=subprocess.PIPE)
stdout, stderr = proc.communicate() stdout, stderr = proc.communicate()
@@ -1046,7 +1046,7 @@ def enable_mod(mod_name, apache_init_script, apache_enmod):
try: try:
# Use check_output so the command will finish before reloading # Use check_output so the command will finish before reloading
# TODO: a2enmod is debian specific... # TODO: a2enmod is debian specific...
subprocess.check_call(["sudo", apache_enmod, mod_name], # TODO: sudo? subprocess.check_call([apache_enmod, mod_name],
stdout=open("/dev/null", "w"), stdout=open("/dev/null", "w"),
stderr=open("/dev/null", "w")) stderr=open("/dev/null", "w"))
apache_restart(apache_init_script) apache_restart(apache_init_script)