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

Added comment to clarify FQDN regex

Browse Source
This commit is contained in:
Joona Hoikkala
2015-11-08 21:14:12 +02:00
parent e63aaf7faf
commit c80e4c0f2f
1 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
letsencrypt/cli.py
+2 -1
View File
@@ -1158,7 +1158,8 @@ def check_config_sanity(args):
# Punycode # Punycode
if any("xn--" in d for d in args.domains): if any("xn--" in d for d in args.domains):
raise errors.ConfigurationError("Error: Punycode domains are not supported") raise errors.ConfigurationError("Error: Punycode domains are not supported")
# Check for FQDN # FQDN, checks:
# Characters used, domain parts < 63 chars, tld > 3 < 6 chars
fqdn = re.compile("^((?!-)[A-Za-z0-9-]{1,63}(?<!-)\\.)+[A-Za-z]{2,6}$") fqdn = re.compile("^((?!-)[A-Za-z0-9-]{1,63}(?<!-)\\.)+[A-Za-z]{2,6}$")
if any(True for d in args.domains if not fqdn.match(d)): if any(True for d in args.domains if not fqdn.match(d)):
raise errors.ConfigurationError("Error: Requested domain is not FQDN") raise errors.ConfigurationError("Error: Requested domain is not FQDN")