From 2ce7d5cbd636b5976f5e1aa00464982d73daf6dc Mon Sep 17 00:00:00 2001
From: Noah Swartz <noah@eff.org>
Date: Thu, 17 Dec 2015 12:22:09 -0800
Subject: [PATCH 1/4] add support for verbose count setting logger level

---
 letsencrypt/storage.py | 17 +++++++++++++++++
 1 file changed, 17 insertions(+)

diff --git a/letsencrypt/storage.py b/letsencrypt/storage.py
index 3b2b548b0..9614f091a 100644
--- a/letsencrypt/storage.py
+++ b/letsencrypt/storage.py
@@ -116,6 +116,8 @@ class RenewableCert(object):  # pylint: disable=too-many-instance-attributes
         #       read further defaults from the systemwide renewal configuration
         #       file at this stage?
         self.configuration = config_with_defaults(self.configfile)
+        logger_level = self.configuration['renewalparams']['verbose_count']
+        set_logger_level(logger_level)
 
         if not all(x in self.configuration for x in ALL_FOUR):
             raise errors.CertStorageError(
@@ -129,6 +131,21 @@ class RenewableCert(object):  # pylint: disable=too-many-instance-attributes
 
         self._fix_symlinks()
 
+    def set_logger_level(logger_level):
+        levels_dict = {"0" : 0,
+                       "-1" : 10,
+                       "-2" : 20,
+                       "-3" : 30,
+                       "-4" : 40,
+                       "-5" : 50}
+        if logger_level in levels_dict:
+            new_level = levels_dict[logger_level]
+        else:
+            new_level = 30
+        root_logger = logger.parent
+        root_logger.setLevel(new_level)
+        return
+
     def _consistent(self):
         """Are the files associated with this lineage self-consistent?
 

From 44a9d3d2907a3dc87c2536e2620947d9711ad879 Mon Sep 17 00:00:00 2001
From: Noah Swartz <noah@eff.org>
Date: Thu, 17 Dec 2015 12:29:28 -0800
Subject: [PATCH 2/4] fixed self issue

---
 letsencrypt/storage.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/letsencrypt/storage.py b/letsencrypt/storage.py
index 9614f091a..c79903039 100644
--- a/letsencrypt/storage.py
+++ b/letsencrypt/storage.py
@@ -117,7 +117,7 @@ class RenewableCert(object):  # pylint: disable=too-many-instance-attributes
         #       file at this stage?
         self.configuration = config_with_defaults(self.configfile)
         logger_level = self.configuration['renewalparams']['verbose_count']
-        set_logger_level(logger_level)
+        self.set_logger_level(logger_level)
 
         if not all(x in self.configuration for x in ALL_FOUR):
             raise errors.CertStorageError(
@@ -131,7 +131,7 @@ class RenewableCert(object):  # pylint: disable=too-many-instance-attributes
 
         self._fix_symlinks()
 
-    def set_logger_level(logger_level):
+    def set_logger_level(self, logger_level):
         levels_dict = {"0" : 0,
                        "-1" : 10,
                        "-2" : 20,

From 253cc3dc8f0b30a2aaa1d5b2ae29ea635ccf4b59 Mon Sep 17 00:00:00 2001
From: Noah Swartz <noah@eff.org>
Date: Thu, 17 Dec 2015 14:36:53 -0800
Subject: [PATCH 3/4] have the handler actually set the level of the logger

---
 letsencrypt/renewer.py | 1 +
 1 file changed, 1 insertion(+)

diff --git a/letsencrypt/renewer.py b/letsencrypt/renewer.py
index 8cb5d1c3d..2f36e7e91 100644
--- a/letsencrypt/renewer.py
+++ b/letsencrypt/renewer.py
@@ -116,6 +116,7 @@ def renew(cert, old_version):
 def _cli_log_handler(args, level, fmt):  # pylint: disable=unused-argument
     handler = colored_logging.StreamHandler()
     handler.setFormatter(logging.Formatter(fmt))
+    handler.setLevel(level)
     return handler
 
 

From 79432fddc3cc97d2ca7ca7f525eac9ec76441b30 Mon Sep 17 00:00:00 2001
From: Noah Swartz <noah@eff.org>
Date: Thu, 17 Dec 2015 16:40:56 -0800
Subject: [PATCH 4/4] undo previous logger changes

---
 letsencrypt/storage.py | 17 -----------------
 1 file changed, 17 deletions(-)

diff --git a/letsencrypt/storage.py b/letsencrypt/storage.py
index b9587b909..c2992bb47 100644
--- a/letsencrypt/storage.py
+++ b/letsencrypt/storage.py
@@ -116,8 +116,6 @@ class RenewableCert(object):  # pylint: disable=too-many-instance-attributes
         #       read further defaults from the systemwide renewal configuration
         #       file at this stage?
         self.configuration = config_with_defaults(self.configfile)
-        logger_level = self.configuration['renewalparams']['verbose_count']
-        self.set_logger_level(logger_level)
 
         if not all(x in self.configuration for x in ALL_FOUR):
             raise errors.CertStorageError(
@@ -131,21 +129,6 @@ class RenewableCert(object):  # pylint: disable=too-many-instance-attributes
 
         self._fix_symlinks()
 
-    def set_logger_level(self, logger_level):
-        levels_dict = {"0" : 0,
-                       "-1" : 10,
-                       "-2" : 20,
-                       "-3" : 30,
-                       "-4" : 40,
-                       "-5" : 50}
-        if logger_level in levels_dict:
-            new_level = levels_dict[logger_level]
-        else:
-            new_level = 30
-        root_logger = logger.parent
-        root_logger.setLevel(new_level)
-        return
-
     def _consistent(self):
         """Are the files associated with this lineage self-consistent?