diff --git a/certbot-apache/certbot_apache/_internal/configurator.py b/certbot-apache/certbot_apache/_internal/configurator.py index c24a646db..2f8665a3e 100644 --- a/certbot-apache/certbot_apache/_internal/configurator.py +++ b/certbot-apache/certbot_apache/_internal/configurator.py @@ -1838,13 +1838,13 @@ class ApacheConfigurator(common.Installer): if options: msg_enhancement += ": " + options msg = msg_tmpl.format(domain, msg_enhancement) - logger.warning(msg) + logger.error(msg) raise errors.PluginError(msg) try: for vhost in vhosts: func(vhost, options) except errors.PluginError: - logger.warning("Failed %s for %s", enhancement, domain) + logger.error("Failed %s for %s", enhancement, domain) raise def _autohsts_increase(self, vhost, id_str, nextstep): @@ -2408,7 +2408,7 @@ class ApacheConfigurator(common.Installer): try: util.run_script(self.option("restart_cmd")) except errors.SubprocessError as err: - logger.info("Unable to restart apache using %s", + logger.warning("Unable to restart apache using %s", self.option("restart_cmd")) alt_restart = self.option("restart_cmd_alt") if alt_restart: @@ -2566,7 +2566,7 @@ class ApacheConfigurator(common.Installer): msg_tmpl = ("Certbot was not able to find SSL VirtualHost for a " "domain {0} for enabling AutoHSTS enhancement.") msg = msg_tmpl.format(d) - logger.warning(msg) + logger.error(msg) raise errors.PluginError(msg) for vh in vhosts: try: @@ -2652,7 +2652,7 @@ class ApacheConfigurator(common.Installer): except errors.PluginError: msg = ("Could not find VirtualHost with ID {0}, disabling " "AutoHSTS for this VirtualHost").format(id_str) - logger.warning(msg) + logger.error(msg) # Remove the orphaned AutoHSTS entry from pluginstorage self._autohsts.pop(id_str) continue @@ -2692,7 +2692,7 @@ class ApacheConfigurator(common.Installer): except errors.PluginError: msg = ("VirtualHost with id {} was not found, unable to " "make HSTS max-age permanent.").format(id_str) - logger.warning(msg) + logger.error(msg) self._autohsts.pop(id_str) continue if self._autohsts_vhost_in_lineage(vhost, lineage): diff --git a/certbot-apache/certbot_apache/_internal/display_ops.py b/certbot-apache/certbot_apache/_internal/display_ops.py index dabf20606..875225eb9 100644 --- a/certbot-apache/certbot_apache/_internal/display_ops.py +++ b/certbot-apache/certbot_apache/_internal/display_ops.py @@ -119,7 +119,7 @@ def _vhost_menu(domain, vhosts): "guidance in non-interactive mode. Certbot may need " "vhosts to be explicitly labelled with ServerName or " "ServerAlias directives.".format(domain)) - logger.warning(msg) + logger.error(msg) raise errors.MissingCommandlineFlag(msg) return code, tag diff --git a/certbot-apache/certbot_apache/_internal/override_debian.py b/certbot-apache/certbot_apache/_internal/override_debian.py index 9f938046b..156d979e2 100644 --- a/certbot-apache/certbot_apache/_internal/override_debian.py +++ b/certbot-apache/certbot_apache/_internal/override_debian.py @@ -68,7 +68,7 @@ class DebianConfigurator(configurator.ApacheConfigurator): # Already in shape vhost.enabled = True return None - logger.warning( + logger.error( "Could not symlink %s to %s, got error: %s", enabled_path, vhost.filep, err.strerror) errstring = ("Encountered error while trying to enable a " +