remove check for Redirect header; the existence of a Redirect header does not imply a HTTP->HTTPS redirection

letsencrypt-apache/letsencrypt_apache/configurator.py

@@ -912,11 +912,7 @@ class ApacheConfigurator(augeas_configurator.AugeasConfigurator):
         """
         rewrite_path = self.parser.find_dir(
             "RewriteRule", None, start=vhost.path)
-        redirect_path = self.parser.find_dir("Redirect", None, start=vhost.path)
 
-        if redirect_path:
-            # "Existing Redirect directive for virtualhost"
-            raise errors.PluginError("Existing Redirect present on HTTP vhost.")
         if rewrite_path:
             # "No existing redirection for virtualhost"
             if len(rewrite_path) != len(constants.REWRITE_HTTPS_ARGS):

letsencrypt-apache/letsencrypt_apache/tests/configurator_test.py

@@ -771,16 +771,6 @@ class TwoVhost80Test(util.ApacheTest):
             errors.PluginError,
             self.config.enhance, "letsencrypt.demo", "redirect")
 
-    def test_unknown_redirect(self):
-        # Skip the enable mod
-        self.config.parser.modules.add("rewrite_module")
-        self.config.parser.add_dir(
-            self.vh_truth[3].path, "Redirect", ["Unknown"])
-        self.config.save()
-        self.assertRaises(
-            errors.PluginError,
-            self.config.enhance, "letsencrypt.demo", "redirect")
-
     def test_create_own_redirect(self):
         self.config.parser.modules.add("rewrite_module")
         # For full testing... give names...