fixed linting and added logger

DerekK19 · Dec 24, 2015 · ea2c86b · ea2c86b
1 parent 49f36f8
commit ea2c86b
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/letsencrypt-apache/letsencrypt_apache/configurator.py b/letsencrypt-apache/letsencrypt_apache/configurator.py
@@ -1271,7 +1271,7 @@ def restart(self):
 
         """
         self.config_test()
-        logger.debug(self.aug.view_config_changes(self))
+        logger.debug(self.reverter.view_config_changes())
         self._reload()
 
     def _reload(self):

diff --git a/letsencrypt-apache/letsencrypt_apache/tls_sni_01.py b/letsencrypt-apache/letsencrypt_apache/tls_sni_01.py
@@ -1,12 +1,14 @@
 """A class that performs TLS-SNI-01 challenges for Apache"""
 
 import os
+import logging
 
 from letsencrypt.plugins import common
 
 from letsencrypt_apache import obj
 from letsencrypt_apache import parser
 
+logger = logging.getLogger(__name__)
 
 class ApacheTlsSni01(common.TLSSNI01):
     """Class that performs TLS-SNI-01 challenges within the Apache configurator
@@ -104,9 +106,9 @@ def _mod_config(self):
         self.configurator.reverter.register_file_creation(
             True, self.challenge_conf)
 
-        with open(self.challenge_conf, "rw") as new_conf:
+        logger.debug("writing a config file with text: %s", config_text)
+        with open(self.challenge_conf, "w") as new_conf:
             new_conf.write(config_text)
-            logger.debug(new_conf.read())
 
         return addrs