starred/certbot
1
0
Fork 0
mirror of https://github.com/certbot/certbot.git synced 2026-07-28 16:54:56 +02:00
Code Issues Packages Projects Releases Wiki Activity

Logic for setting configurators assumed they were really being set

Browse Source 
(They might not be)
This commit is contained in:
Peter Eckersley
2015-10-16 17:38:17 -07:00
parent 3d6ecc114b
commit dcf8ea4e97
1 changed files with 3 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
letsencrypt/cli.py
+3
View File
@@ -309,6 +309,9 @@ class ConfiguratorError(TypeError): # pylint: disable=missing-docstring
def set_configurator(previously, now): def set_configurator(previously, now):
"""Setting configurators multiple ways is okay, as long as they all agree""" """Setting configurators multiple ways is okay, as long as they all agree"""
if now is None:
# we're not actually setting anything
return previously
if previously: if previously:
if previously != now: if previously != now:
msg = "Too many flags setting configurators/installers/authenticators %s -> %s" msg = "Too many flags setting configurators/installers/authenticators %s -> %s"