upstream/certbot
1
0
Fork 0
mirror of https://github.com/certbot/certbot.git synced 2026-05-28 04:34:11 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions 2

Don't use sudo in apache plugin.

Browse source
This commit is contained in:
Jakub Warmuz 2015-05-02 08:08:11 +00:00
parent 828e09543b
commit 7a4d37e320
No known key found for this signature in database
GPG key ID: 2A7BAD3A489B52EA
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
letsencrypt/client/plugins/apache/configurator.py
View file

@ -908,7 +908,7 @@ class ApacheConfigurator(augeas_configurator.AugeasConfigurator):
"""
try:
proc = subprocess.Popen(
["sudo", self.config.apache_ctl, "configtest"], # TODO: sudo?
[self.config.apache_ctl, "configtest"],
stdout=subprocess.PIPE,
stderr=subprocess.PIPE)
stdout, stderr = proc.communicate()
@ -1046,7 +1046,7 @@ def enable_mod(mod_name, apache_init_script, apache_enmod):
try:
# Use check_output so the command will finish before reloading
# 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"),
stderr=open("/dev/null", "w"))
apache_restart(apache_init_script)