From 5192fa36ab25fe7a5dbeb757b1d5308e20d7c754 Mon Sep 17 00:00:00 2001
From: Noah Swartz <noah@eff.org>
Date: Fri, 22 Jan 2016 11:47:49 -0800
Subject: [PATCH] move save command up to tls_sni

---
 letsencrypt-apache/letsencrypt_apache/configurator.py | 7 ++-----
 letsencrypt-apache/letsencrypt_apache/tls_sni_01.py   | 1 +
 2 files changed, 3 insertions(+), 5 deletions(-)

diff --git a/letsencrypt-apache/letsencrypt_apache/configurator.py b/letsencrypt-apache/letsencrypt_apache/configurator.py
index 65e7a14a8..8818923f4 100644
--- a/letsencrypt-apache/letsencrypt_apache/configurator.py
+++ b/letsencrypt-apache/letsencrypt_apache/configurator.py
@@ -557,11 +557,8 @@ class ApacheConfigurator(augeas_configurator.AugeasConfigurator):
 
         # search for NameVirtualHost directive for ip_addr
         # note ip_addr can be FQDN although Apache does not recommend it
-        if self.version >= (2, 4):
-            return True
-        else:
-            self.save("don't lose config changes", True)
-            return self.parser.find_dir("NameVirtualHost", str(target_addr))
+        return (self.version >= (2, 4) or
+                self.parser.find_dir("NameVirtualHost", str(target_addr)))
 
     def add_name_vhost(self, addr):
         """Adds NameVirtualHost directive for given address.
diff --git a/letsencrypt-apache/letsencrypt_apache/tls_sni_01.py b/letsencrypt-apache/letsencrypt_apache/tls_sni_01.py
index 971072311..cc1d749a0 100644
--- a/letsencrypt-apache/letsencrypt_apache/tls_sni_01.py
+++ b/letsencrypt-apache/letsencrypt_apache/tls_sni_01.py
@@ -76,6 +76,7 @@ class ApacheTlsSni01(common.TLSSNI01):
 
         # Setup the configuration
         addrs = self._mod_config()
+        self.configurator.save("Don't lose mod_config changes", True)
         self.configurator.make_addrs_sni_ready(addrs)
 
         # Save reversible changes