diff options
author | ohemorange <ebportnoy@gmail.com> | 2019-11-26 05:53:20 +0300 |
---|---|---|
committer | GitHub <noreply@github.com> | 2019-11-26 05:53:20 +0300 |
commit | 6d1472bf8c75450d0d2318f19ce13539bd644cc0 (patch) | |
tree | 18e809c6a9c0e1b69b755b929f51c4fce7a9a107 | |
parent | 5c8083851a230df42eb4493020086eef51ab073e (diff) |
Implement redirect by default (#7595)
* Change redirect default to yes so that it happens automatically in noninteractive mode
* Update changelog
-rw-r--r-- | certbot/CHANGELOG.md | 3 | ||||
-rw-r--r-- | certbot/certbot/_internal/display/enhancements.py | 2 |
2 files changed, 4 insertions, 1 deletions
diff --git a/certbot/CHANGELOG.md b/certbot/CHANGELOG.md index dfb6acde8..b5794c2dd 100644 --- a/certbot/CHANGELOG.md +++ b/certbot/CHANGELOG.md @@ -28,6 +28,9 @@ Certbot adheres to [Semantic Versioning](https://semver.org/). `certbot.reverter.Reverter.view_config_changes`, and `certbot.util.get_systemd_os_info` have been removed * Certbot's `register --update-registration` subcommand has been removed +* When possible, default to automatically configuring the webserver so all requests + redirect to secure HTTPS access. This is mostly relevant when running Certbot + in non-interactive mode. Previously, the default was to not redirect all requests. ### Fixed diff --git a/certbot/certbot/_internal/display/enhancements.py b/certbot/certbot/_internal/display/enhancements.py index 5498b9547..0529f53c6 100644 --- a/certbot/certbot/_internal/display/enhancements.py +++ b/certbot/certbot/_internal/display/enhancements.py @@ -50,7 +50,7 @@ def redirect_by_default(): code, selection = util(interfaces.IDisplay).menu( "Please choose whether or not to redirect HTTP traffic to HTTPS, removing HTTP access.", - choices, default=0, + choices, default=1, cli_flag="--redirect / --no-redirect", force_interactive=True) if code != display_util.OK: |