Switch to using defaultdict(list)

1 files changed, 3 insertions, 5 deletions

diff --git a/letsencrypt-apache/letsencrypt_apache/configurator.py b/letsencrypt-apache/letsencrypt_apache/configurator.py
index bc12a75fe..6b4a0e46e 100644
--- a/letsencrypt-apache/letsencrypt_apache/configurator.py
+++ b/letsencrypt-apache/letsencrypt_apache/configurator.py
@@ -25,6 +25,7 @@ from letsencrypt_apache import tls_sni_01
 from letsencrypt_apache import obj
 from letsencrypt_apache import parser
 
+from collections import defaultdict
 
 logger = logging.getLogger(__name__)
 
@@ -930,16 +931,13 @@ class ApacheConfigurator(augeas_configurator.AugeasConfigurator):
         rewrite_path = self.parser.find_dir(
                 "RewriteRule", None, start=vhost.path)
 
-        dir_dict = {}
+        dir_dict = defaultdict(list)
         pat = r'.*(directive\[\d+\]).*'
         for match in rewrite_path:
             m = re.match(pat, match)
             if m:
                 dir_id = m.group(1)
-                if dir_id in dir_dict:
-                    dir_dict[dir_id].append(match)
-                else:
-                    dir_dict[dir_id] = [match]
+                dir_dict[dir_id].append(match)
 
         if dir_dict:
             for dir_id in dir_dict: