diff options
author | Will Greenberg <willg@eff.org> | 2022-09-07 05:55:58 +0300 |
---|---|---|
committer | GitHub <noreply@github.com> | 2022-09-07 05:55:58 +0300 |
commit | 20ca9288d5c55f47a9dbbafe3b9fe4157308cb77 (patch) | |
tree | 20e59eae89922b9cae8924f751f41166dcd4f98c | |
parent | c20d40ddbafb8db9e38cf5ec41b0ec187cc06686 (diff) |
Add UI text recommending multi-domain certs (#9393)
* Suggest multi-domain certs in domain selection menu
* Update changelog
* lint: fix long line
Co-authored-by: Alex Zorin <alex@zorin.id.au>
-rw-r--r-- | certbot/CHANGELOG.md | 1 | ||||
-rw-r--r-- | certbot/certbot/display/ops.py | 5 |
2 files changed, 5 insertions, 1 deletions
diff --git a/certbot/CHANGELOG.md b/certbot/CHANGELOG.md index 22c2ffdba..06a415777 100644 --- a/certbot/CHANGELOG.md +++ b/certbot/CHANGELOG.md @@ -23,6 +23,7 @@ Certbot adheres to [Semantic Versioning](https://semver.org/). next major release of Certbot. * The `source_address` argument for `acme.client.ClientNetwork` is deprecated and support for it will be removed in the next major release. +* Add UI text suggesting users create certs for multiple domains, when possible ### Fixed diff --git a/certbot/certbot/display/ops.py b/certbot/certbot/display/ops.py index f07093b55..d5b2c2420 100644 --- a/certbot/certbot/display/ops.py +++ b/certbot/certbot/display/ops.py @@ -181,7 +181,10 @@ def _filter_names(names: Iterable[str], if override_question: question = override_question else: - question = "Which names would you like to activate HTTPS for?" + question = ( + "Which names would you like to activate HTTPS for?\n" + "We recommend selecting either all domains, or all domains in a VirtualHost/server " + "block.") code, names = display_util.checklist( question, tags=sorted_names, cli_flag="--domains", force_interactive=True) return code, [str(s) for s in names] |