starred/certbot-certbot
1
0
Fork 0
mirror of https://github.com/certbot/certbot.git synced 2026-08-02 19:31:51 +02:00
Code Issues Packages Projects Releases Wiki Activity

Show error details for the nginx config parser (#4221)

Browse Source 
Nginx parser errors now include helpful details on where parsing has
actually failed.

Related: https://github.com/certbot/certbot/issues/3798
This commit is contained in:
silverwind
2017-02-22 18:50:56 -08:00
committed by Brad Warren
parent a92ca8e97c
commit b1a4280519
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
certbot-nginx/certbot_nginx/parser.py
+4 -4
View File
@@ -205,8 +205,8 @@ class NginxParser(object):
trees.append(parsed) trees.append(parsed)
except IOError: except IOError:
logger.warning("Could not open file: %s", item) logger.warning("Could not open file: %s", item)
except pyparsing.ParseException: except pyparsing.ParseException as err:
logger.debug("Could not parse file: %s", item) logger.debug("Could not parse file: %s due to %s", item, err)
return trees return trees
def _parse_ssl_options(self, ssl_options): def _parse_ssl_options(self, ssl_options):
@@ -216,8 +216,8 @@ class NginxParser(object):
return nginxparser.load(_file).spaced return nginxparser.load(_file).spaced
except IOError: except IOError:
logger.warn("Missing NGINX TLS options file: %s", ssl_options) logger.warn("Missing NGINX TLS options file: %s", ssl_options)
except pyparsing.ParseBaseException: except pyparsing.ParseBaseException as err:
logger.debug("Could not parse file: %s", ssl_options) logger.debug("Could not parse file: %s due to %s", ssl_options, err)
return [] return []
def _set_locations(self, ssl_options): def _set_locations(self, ssl_options):