From b1a4280519b1a2d271e0e770a87a2a37915a381a Mon Sep 17 00:00:00 2001 From: silverwind Date: Thu, 23 Feb 2017 03:50:56 +0100 Subject: [PATCH] Show error details for the nginx config parser (#4221) Nginx parser errors now include helpful details on where parsing has actually failed. Related: https://github.com/certbot/certbot/issues/3798 --- certbot-nginx/certbot_nginx/parser.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/certbot-nginx/certbot_nginx/parser.py b/certbot-nginx/certbot_nginx/parser.py index 1a2c85c2c..c586aa459 100644 --- a/certbot-nginx/certbot_nginx/parser.py +++ b/certbot-nginx/certbot_nginx/parser.py @@ -205,8 +205,8 @@ class NginxParser(object): trees.append(parsed) except IOError: logger.warning("Could not open file: %s", item) - except pyparsing.ParseException: - logger.debug("Could not parse file: %s", item) + except pyparsing.ParseException as err: + logger.debug("Could not parse file: %s due to %s", item, err) return trees def _parse_ssl_options(self, ssl_options): @@ -216,8 +216,8 @@ class NginxParser(object): return nginxparser.load(_file).spaced except IOError: logger.warn("Missing NGINX TLS options file: %s", ssl_options) - except pyparsing.ParseBaseException: - logger.debug("Could not parse file: %s", ssl_options) + except pyparsing.ParseBaseException as err: + logger.debug("Could not parse file: %s due to %s", ssl_options, err) return [] def _set_locations(self, ssl_options):