upstream/certbot
1
0
Fork 0
mirror of https://github.com/certbot/certbot.git synced 2026-05-28 04:34:11 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions 2

New _get_names approach for nginx test

Browse source
This commit is contained in:
Seth Schoen 2016-08-03 17:10:20 -07:00
parent 7b67ba6797
commit 353cb6e6c6
1 changed files with 7 additions and 20 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

27
certbot-compatibility-test/certbot_compatibility_test/configurators/nginx/common.py
View file

@ -125,25 +125,12 @@ def _get_server_root(config):
def _get_names(config):
"""Returns all and testable domain names in config"""
# XXX: This is still Apache-specific
all_names = set()
non_ip_names = set()
with open(os.path.join(config, "vhosts")) as f:
for line in f:
# If parsing a specific vhost
if line[0].isspace():
words = line.split()
if words[0] == "alias":
all_names.add(words[1])
non_ip_names.add(words[1])
# If for port 80 and not IP vhost
elif words[1] == "80" and not util.IP_REGEX.match(words[3]):
all_names.add(words[3])
non_ip_names.add(words[3])
elif "NameVirtualHost" not in line:
words = line.split()
if (words[0].endswith("*") or words[0].endswith("80") and
not util.IP_REGEX.match(words[1]) and
words[1].find(".") != -1):
all_names.add(words[1])
for root, _dirs, files in os.walk(config):
for this_file in files:
for line in open(os.path.join(root, this_file)):
if line.strip().starts_with("server_name"):
names = line.partition("server_name")[2].rstrip(";")
[all_names.add(n) for n in names.split()]
non_ip_names = set(n for n in all_names if not util.IP_REGEX.match(n))
return all_names, non_ip_names