2019-04-11 21:07:36 -04:00
|
|
|
"""Module executing integration tests against certbot core."""
|
|
|
|
|
from __future__ import print_function
|
2019-04-15 17:59:45 -04:00
|
|
|
|
2019-04-11 21:07:36 -04:00
|
|
|
import os
|
2019-04-15 19:42:06 -04:00
|
|
|
import re
|
|
|
|
|
import shutil
|
2019-04-15 17:59:45 -04:00
|
|
|
import subprocess
|
2019-05-14 16:56:32 -04:00
|
|
|
import time
|
2019-04-15 19:42:06 -04:00
|
|
|
from os.path import join, exists
|
2019-03-01 16:18:06 -05:00
|
|
|
|
|
|
|
|
import pytest
|
|
|
|
|
from certbot_integration_tests.certbot_tests import context as certbot_context
|
2019-04-11 21:07:36 -04:00
|
|
|
from certbot_integration_tests.certbot_tests.assertions import (
|
2019-08-06 18:02:16 -04:00
|
|
|
assert_hook_execution, assert_saved_renew_hook,
|
|
|
|
|
assert_cert_count_for_lineage,
|
|
|
|
|
assert_world_no_permissions, assert_world_read_permissions,
|
|
|
|
|
assert_equals_group_owner, assert_equals_group_permissions, assert_equals_world_read_permissions,
|
|
|
|
|
EVERYBODY_SID
|
2019-04-11 21:07:36 -04:00
|
|
|
)
|
|
|
|
|
from certbot_integration_tests.utils import misc
|
2019-03-01 16:18:06 -05:00
|
|
|
|
|
|
|
|
|
|
|
|
|
@pytest.fixture()
|
|
|
|
|
def context(request):
|
|
|
|
|
# Fixture request is a built-in pytest fixture describing current test request.
|
|
|
|
|
integration_test_context = certbot_context.IntegrationTestsContext(request)
|
|
|
|
|
try:
|
|
|
|
|
yield integration_test_context
|
|
|
|
|
finally:
|
|
|
|
|
integration_test_context.cleanup()
|
|
|
|
|
|
|
|
|
|
|
2019-04-15 17:59:45 -04:00
|
|
|
def test_basic_commands(context):
|
|
|
|
|
"""Test simple commands on Certbot CLI."""
|
|
|
|
|
# TMPDIR env variable is set to workspace for the certbot subprocess.
|
|
|
|
|
# So tempdir module will create any temporary files/dirs in workspace,
|
|
|
|
|
# and its content can be tested to check correct certbot cleanup.
|
|
|
|
|
initial_count_tmpfiles = len(os.listdir(context.workspace))
|
|
|
|
|
|
|
|
|
|
context.certbot(['--help'])
|
|
|
|
|
context.certbot(['--help', 'all'])
|
|
|
|
|
context.certbot(['--version'])
|
|
|
|
|
|
|
|
|
|
with pytest.raises(subprocess.CalledProcessError):
|
|
|
|
|
context.certbot(['--csr'])
|
|
|
|
|
|
|
|
|
|
new_count_tmpfiles = len(os.listdir(context.workspace))
|
|
|
|
|
assert initial_count_tmpfiles == new_count_tmpfiles
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
def test_hook_dirs_creation(context):
|
|
|
|
|
"""Test all hooks directory are created during Certbot startup."""
|
|
|
|
|
context.certbot(['register'])
|
|
|
|
|
|
|
|
|
|
for hook_dir in misc.list_renewal_hooks_dirs(context.config_dir):
|
|
|
|
|
assert os.path.isdir(hook_dir)
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
def test_registration_override(context):
|
|
|
|
|
"""Test correct register/unregister, and registration override."""
|
|
|
|
|
context.certbot(['register'])
|
|
|
|
|
context.certbot(['unregister'])
|
|
|
|
|
context.certbot(['register', '--email', 'ex1@domain.org,ex2@domain.org'])
|
|
|
|
|
|
|
|
|
|
# TODO: When `certbot register --update-registration` is fully deprecated,
|
|
|
|
|
# delete the two following deprecated uses
|
|
|
|
|
context.certbot(['register', '--update-registration', '--email', 'ex1@domain.org'])
|
|
|
|
|
context.certbot(['register', '--update-registration', '--email', 'ex1@domain.org,ex2@domain.org'])
|
|
|
|
|
|
|
|
|
|
context.certbot(['update_account', '--email', 'example@domain.org'])
|
|
|
|
|
context.certbot(['update_account', '--email', 'ex1@domain.org,ex2@domain.org'])
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
def test_prepare_plugins(context):
|
|
|
|
|
"""Test that plugins are correctly instantiated and displayed."""
|
|
|
|
|
output = context.certbot(['plugins', '--init', '--prepare'])
|
|
|
|
|
|
|
|
|
|
assert 'webroot' in output
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
def test_http_01(context):
|
|
|
|
|
"""Test the HTTP-01 challenge using standalone plugin."""
|
|
|
|
|
# We start a server listening on the port for the
|
|
|
|
|
# TLS-SNI challenge to prevent regressions in #3601.
|
|
|
|
|
with misc.create_http_server(context.tls_alpn_01_port):
|
|
|
|
|
certname = context.get_domain('le2')
|
|
|
|
|
context.certbot([
|
|
|
|
|
'--domains', certname, '--preferred-challenges', 'http-01', 'run',
|
|
|
|
|
'--cert-name', certname,
|
2019-08-06 18:02:16 -04:00
|
|
|
'--pre-hook', misc.echo('wtf_pre', context.hook_probe),
|
|
|
|
|
'--post-hook', misc.echo('wtf_post', context.hook_probe),
|
|
|
|
|
'--deploy-hook', misc.echo('deploy', context.hook_probe),
|
2019-04-15 17:59:45 -04:00
|
|
|
])
|
|
|
|
|
|
|
|
|
|
assert_hook_execution(context.hook_probe, 'deploy')
|
|
|
|
|
assert_saved_renew_hook(context.config_dir, certname)
|
|
|
|
|
|
|
|
|
|
|
2019-04-11 21:07:36 -04:00
|
|
|
def test_manual_http_auth(context):
|
|
|
|
|
"""Test the HTTP-01 challenge using manual plugin."""
|
|
|
|
|
with misc.create_http_server(context.http_01_port) as webroot,\
|
|
|
|
|
misc.manual_http_hooks(webroot, context.http_01_port) as scripts:
|
2019-03-01 16:18:06 -05:00
|
|
|
|
2019-04-11 21:07:36 -04:00
|
|
|
certname = context.get_domain()
|
|
|
|
|
context.certbot([
|
|
|
|
|
'certonly', '-a', 'manual', '-d', certname,
|
|
|
|
|
'--cert-name', certname,
|
|
|
|
|
'--manual-auth-hook', scripts[0],
|
|
|
|
|
'--manual-cleanup-hook', scripts[1],
|
2019-08-06 18:02:16 -04:00
|
|
|
'--pre-hook', misc.echo('wtf_pre', context.hook_probe),
|
|
|
|
|
'--post-hook', misc.echo('wtf_post', context.hook_probe),
|
|
|
|
|
'--renew-hook', misc.echo('renew', context.hook_probe),
|
2019-04-11 21:07:36 -04:00
|
|
|
])
|
2019-03-01 16:18:06 -05:00
|
|
|
|
2019-04-15 17:59:45 -04:00
|
|
|
with pytest.raises(AssertionError):
|
|
|
|
|
assert_hook_execution(context.hook_probe, 'renew')
|
|
|
|
|
assert_saved_renew_hook(context.config_dir, certname)
|
2019-04-11 21:07:36 -04:00
|
|
|
|
|
|
|
|
|
|
|
|
|
def test_manual_dns_auth(context):
|
|
|
|
|
"""Test the DNS-01 challenge using manual plugin."""
|
|
|
|
|
certname = context.get_domain('dns')
|
|
|
|
|
context.certbot([
|
|
|
|
|
'-a', 'manual', '-d', certname, '--preferred-challenges', 'dns',
|
|
|
|
|
'run', '--cert-name', certname,
|
|
|
|
|
'--manual-auth-hook', context.manual_dns_auth_hook,
|
|
|
|
|
'--manual-cleanup-hook', context.manual_dns_cleanup_hook,
|
2019-08-06 18:02:16 -04:00
|
|
|
'--pre-hook', misc.echo('wtf_pre', context.hook_probe),
|
|
|
|
|
'--post-hook', misc.echo('wtf_post', context.hook_probe),
|
|
|
|
|
'--renew-hook', misc.echo('renew', context.hook_probe),
|
2019-04-11 21:07:36 -04:00
|
|
|
])
|
|
|
|
|
|
|
|
|
|
with pytest.raises(AssertionError):
|
|
|
|
|
assert_hook_execution(context.hook_probe, 'renew')
|
2019-04-15 17:59:45 -04:00
|
|
|
assert_saved_renew_hook(context.config_dir, certname)
|
2019-04-11 21:07:36 -04:00
|
|
|
|
2019-04-15 19:18:24 -04:00
|
|
|
context.certbot(['renew', '--cert-name', certname, '--authenticator', 'manual'])
|
|
|
|
|
|
|
|
|
|
assert_cert_count_for_lineage(context.config_dir, certname, 2)
|
|
|
|
|
|
2019-04-11 21:07:36 -04:00
|
|
|
|
2019-04-15 18:04:22 -04:00
|
|
|
def test_certonly(context):
|
|
|
|
|
"""Test the certonly verb on certbot."""
|
|
|
|
|
context.certbot(['certonly', '--cert-name', 'newname', '-d', context.get_domain('newname')])
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
def test_auth_and_install_with_csr(context):
|
|
|
|
|
"""Test certificate issuance and install using an existing CSR."""
|
|
|
|
|
certname = context.get_domain('le3')
|
|
|
|
|
key_path = join(context.workspace, 'key.pem')
|
|
|
|
|
csr_path = join(context.workspace, 'csr.der')
|
|
|
|
|
|
|
|
|
|
misc.generate_csr([certname], key_path, csr_path)
|
|
|
|
|
|
|
|
|
|
cert_path = join(context.workspace, 'csr', 'cert.pem')
|
|
|
|
|
chain_path = join(context.workspace, 'csr', 'chain.pem')
|
|
|
|
|
|
|
|
|
|
context.certbot([
|
|
|
|
|
'auth', '--csr', csr_path,
|
|
|
|
|
'--cert-path', cert_path,
|
|
|
|
|
'--chain-path', chain_path
|
|
|
|
|
])
|
|
|
|
|
|
|
|
|
|
print(misc.read_certificate(cert_path))
|
|
|
|
|
print(misc.read_certificate(chain_path))
|
|
|
|
|
|
|
|
|
|
context.certbot([
|
|
|
|
|
'--domains', certname, 'install',
|
|
|
|
|
'--cert-path', cert_path,
|
|
|
|
|
'--key-path', key_path
|
|
|
|
|
])
|
|
|
|
|
|
|
|
|
|
|
2019-04-11 21:07:36 -04:00
|
|
|
def test_renew_files_permissions(context):
|
2019-04-15 17:59:45 -04:00
|
|
|
"""Test proper certificate file permissions upon renewal"""
|
2019-04-11 21:07:36 -04:00
|
|
|
certname = context.get_domain('renew')
|
|
|
|
|
context.certbot(['-d', certname])
|
|
|
|
|
|
2019-08-06 18:02:16 -04:00
|
|
|
privkey1 = join(context.config_dir, 'archive', certname, 'privkey1.pem')
|
|
|
|
|
privkey2 = join(context.config_dir, 'archive', certname, 'privkey2.pem')
|
|
|
|
|
|
2019-04-11 21:07:36 -04:00
|
|
|
assert_cert_count_for_lineage(context.config_dir, certname, 1)
|
2019-08-06 18:02:16 -04:00
|
|
|
assert_world_no_permissions(privkey1)
|
2019-04-11 21:07:36 -04:00
|
|
|
|
|
|
|
|
context.certbot(['renew'])
|
|
|
|
|
|
|
|
|
|
assert_cert_count_for_lineage(context.config_dir, certname, 2)
|
2019-08-06 18:02:16 -04:00
|
|
|
assert_world_no_permissions(privkey2)
|
|
|
|
|
assert_equals_group_owner(privkey1, privkey2)
|
|
|
|
|
assert_equals_world_read_permissions(privkey1, privkey2)
|
|
|
|
|
assert_equals_group_permissions(privkey1, privkey2)
|
2019-04-11 21:07:36 -04:00
|
|
|
|
|
|
|
|
|
|
|
|
|
def test_renew_with_hook_scripts(context):
|
|
|
|
|
"""Test certificate renewal with script hooks."""
|
|
|
|
|
certname = context.get_domain('renew')
|
|
|
|
|
context.certbot(['-d', certname])
|
|
|
|
|
|
|
|
|
|
assert_cert_count_for_lineage(context.config_dir, certname, 1)
|
|
|
|
|
|
|
|
|
|
misc.generate_test_file_hooks(context.config_dir, context.hook_probe)
|
|
|
|
|
context.certbot(['renew'])
|
|
|
|
|
|
|
|
|
|
assert_cert_count_for_lineage(context.config_dir, certname, 2)
|
|
|
|
|
assert_hook_execution(context.hook_probe, 'deploy')
|
2019-04-15 18:09:57 -04:00
|
|
|
|
|
|
|
|
|
|
|
|
|
def test_renew_files_propagate_permissions(context):
|
|
|
|
|
"""Test proper certificate renewal with custom permissions propagated on private key."""
|
|
|
|
|
certname = context.get_domain('renew')
|
|
|
|
|
context.certbot(['-d', certname])
|
|
|
|
|
|
|
|
|
|
assert_cert_count_for_lineage(context.config_dir, certname, 1)
|
|
|
|
|
|
2019-08-06 18:02:16 -04:00
|
|
|
privkey1 = join(context.config_dir, 'archive', certname, 'privkey1.pem')
|
|
|
|
|
privkey2 = join(context.config_dir, 'archive', certname, 'privkey2.pem')
|
|
|
|
|
|
|
|
|
|
if os.name != 'nt':
|
|
|
|
|
os.chmod(privkey1, 0o444)
|
|
|
|
|
else:
|
|
|
|
|
import win32security
|
|
|
|
|
import ntsecuritycon
|
|
|
|
|
# Get the current DACL of the private key
|
|
|
|
|
security = win32security.GetFileSecurity(privkey1, win32security.DACL_SECURITY_INFORMATION)
|
|
|
|
|
dacl = security.GetSecurityDescriptorDacl()
|
|
|
|
|
# Create a read permission for Everybody group
|
|
|
|
|
everybody = win32security.ConvertStringSidToSid(EVERYBODY_SID)
|
|
|
|
|
dacl.AddAccessAllowedAce(win32security.ACL_REVISION, ntsecuritycon.FILE_GENERIC_READ, everybody)
|
|
|
|
|
# Apply the updated DACL to the private key
|
|
|
|
|
security.SetSecurityDescriptorDacl(1, dacl, 0)
|
|
|
|
|
win32security.SetFileSecurity(privkey1, win32security.DACL_SECURITY_INFORMATION, security)
|
|
|
|
|
|
2019-04-15 18:09:57 -04:00
|
|
|
context.certbot(['renew'])
|
|
|
|
|
|
|
|
|
|
assert_cert_count_for_lineage(context.config_dir, certname, 2)
|
2019-08-06 18:02:16 -04:00
|
|
|
if os.name != 'nt':
|
|
|
|
|
# On Linux, read world permissions + all group permissions will be copied from the previous private key
|
|
|
|
|
assert_world_read_permissions(privkey2)
|
|
|
|
|
assert_equals_world_read_permissions(privkey1, privkey2)
|
|
|
|
|
assert_equals_group_permissions(privkey1, privkey2)
|
|
|
|
|
else:
|
|
|
|
|
# On Windows, world will never have any permissions, and group permission is irrelevant for this platform
|
|
|
|
|
assert_world_no_permissions(privkey2)
|
2019-04-15 18:09:57 -04:00
|
|
|
|
|
|
|
|
|
|
|
|
|
def test_graceful_renew_it_is_not_time(context):
|
|
|
|
|
"""Test graceful renew is not done when it is not due time."""
|
|
|
|
|
certname = context.get_domain('renew')
|
|
|
|
|
context.certbot(['-d', certname])
|
|
|
|
|
|
|
|
|
|
assert_cert_count_for_lineage(context.config_dir, certname, 1)
|
|
|
|
|
|
2019-08-06 18:02:16 -04:00
|
|
|
context.certbot(['renew', '--deploy-hook', misc.echo('deploy', context.hook_probe)],
|
Add executable scripts to start certbot and acme server in certbot-ci (#7073)
During review of #6989, we saw that some of our test bash scripts were still used in the Boulder project in particular. It is about `tests/integration/_common.sh` in particular, to expose the `certbot_test` bash function, that is an appropriate way to execute a local version of certbot in test mode: define a custom server, remove several checks, full log and so on.
This PR is an attempt to assert this goal: exposing a new `certbot_test` executable for test purpose. More generally, this PR is about giving well suited scripts to quickly make manual tests against certbot without launching the full automated pytest suite.
The idea here is to leverage the existing logic in certbot-ci, and expose it as executable scripts. This is done thanks to the `console_scripts` entry of setuptools entrypoint feature, that install scripts in the `PATH`, when `pip install` is invoked, that delegate to specific functions in the installed packages.
Two scripts are defined this way:
* `certbot_test`: it executes certbot in test mode in a very similar way than the original `certbot_test` in `_common.sh`, by delegating to `certbot_integration_tests.utils.certbot_call:main`. By default this execution will target a pebble directory url started locally. The url, and also http-01/tls-alpn-01 challenge ports can be configured using ad-hoc environment variables. All arguments passed to `certbot_test` are transferred to the underlying certbot command.
* `acme_server`: it set up a fully running instance of an ACME server, ready for tests (in particular, all FQDN resolves to localhost in order to target a locally running `certbot_test` command) by delegating to `certbot_integration_tests.utils.acme_server:main`. The choice of the ACME server is given by the first parameter passed to `acme_server`, it can be `pebble`, `boulder-v1` or `boulder-v2`. The command keeps running on foreground, displaying the logs of the ACME server on stdout/stderr. The server is shut down and resources cleaned upon entering CTRL+C.
This two commands can be run also through the underlying python modules, that are executable.
Finally, a typical workflow on certbot side to run manual tests would be:
```
cd certbot
tools/venv.py
source venv/bin/activate
acme_server pebble &
certbot_test certonly --standalone -d test.example.com
```
On boulder side it could be:
```
# Follow certbot dev environment setup instructions, then ...
cd boulder
docker-compose run --use-aliases -e FAKE_DNS=172.17.0.1 --service-ports boulder ./start.py
SERVER=http://localhost:4001/directory certbot_test certonly --standalone -d test.example.com
```
* Configure certbot-ci to expose a certbot_test console script calling certbot in test mode against a local pebble instance
* Add a command to start pebble/boulder
* Use explicit start
* Add execution permission to acme_server
* Add a docstring to certbot_test function
* Change executable name
* Increase sleep to 3600s
* Implement a context manager to handle the acme server
* Add certbot_test workspace in .gitignore
* Add documentation
* Remove one function in context, split logic of certbot_test towards capturing non capturing
* Use an explicit an properly configured ACMEServer as handler.
* Add doc. Put constants.
2019-06-12 20:19:23 -04:00
|
|
|
force_renew=False)
|
2019-04-15 18:09:57 -04:00
|
|
|
|
|
|
|
|
assert_cert_count_for_lineage(context.config_dir, certname, 1)
|
|
|
|
|
with pytest.raises(AssertionError):
|
|
|
|
|
assert_hook_execution(context.hook_probe, 'deploy')
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
def test_graceful_renew_it_is_time(context):
|
|
|
|
|
"""Test graceful renew is done when it is due time."""
|
|
|
|
|
certname = context.get_domain('renew')
|
|
|
|
|
context.certbot(['-d', certname])
|
|
|
|
|
|
|
|
|
|
assert_cert_count_for_lineage(context.config_dir, certname, 1)
|
|
|
|
|
|
|
|
|
|
with open(join(context.config_dir, 'renewal', '{0}.conf'.format(certname)), 'r') as file:
|
|
|
|
|
lines = file.readlines()
|
|
|
|
|
lines.insert(4, 'renew_before_expiry = 100 years{0}'.format(os.linesep))
|
|
|
|
|
with open(join(context.config_dir, 'renewal', '{0}.conf'.format(certname)), 'w') as file:
|
|
|
|
|
file.writelines(lines)
|
|
|
|
|
|
2019-08-06 18:02:16 -04:00
|
|
|
context.certbot(['renew', '--deploy-hook', misc.echo('deploy', context.hook_probe)],
|
Add executable scripts to start certbot and acme server in certbot-ci (#7073)
During review of #6989, we saw that some of our test bash scripts were still used in the Boulder project in particular. It is about `tests/integration/_common.sh` in particular, to expose the `certbot_test` bash function, that is an appropriate way to execute a local version of certbot in test mode: define a custom server, remove several checks, full log and so on.
This PR is an attempt to assert this goal: exposing a new `certbot_test` executable for test purpose. More generally, this PR is about giving well suited scripts to quickly make manual tests against certbot without launching the full automated pytest suite.
The idea here is to leverage the existing logic in certbot-ci, and expose it as executable scripts. This is done thanks to the `console_scripts` entry of setuptools entrypoint feature, that install scripts in the `PATH`, when `pip install` is invoked, that delegate to specific functions in the installed packages.
Two scripts are defined this way:
* `certbot_test`: it executes certbot in test mode in a very similar way than the original `certbot_test` in `_common.sh`, by delegating to `certbot_integration_tests.utils.certbot_call:main`. By default this execution will target a pebble directory url started locally. The url, and also http-01/tls-alpn-01 challenge ports can be configured using ad-hoc environment variables. All arguments passed to `certbot_test` are transferred to the underlying certbot command.
* `acme_server`: it set up a fully running instance of an ACME server, ready for tests (in particular, all FQDN resolves to localhost in order to target a locally running `certbot_test` command) by delegating to `certbot_integration_tests.utils.acme_server:main`. The choice of the ACME server is given by the first parameter passed to `acme_server`, it can be `pebble`, `boulder-v1` or `boulder-v2`. The command keeps running on foreground, displaying the logs of the ACME server on stdout/stderr. The server is shut down and resources cleaned upon entering CTRL+C.
This two commands can be run also through the underlying python modules, that are executable.
Finally, a typical workflow on certbot side to run manual tests would be:
```
cd certbot
tools/venv.py
source venv/bin/activate
acme_server pebble &
certbot_test certonly --standalone -d test.example.com
```
On boulder side it could be:
```
# Follow certbot dev environment setup instructions, then ...
cd boulder
docker-compose run --use-aliases -e FAKE_DNS=172.17.0.1 --service-ports boulder ./start.py
SERVER=http://localhost:4001/directory certbot_test certonly --standalone -d test.example.com
```
* Configure certbot-ci to expose a certbot_test console script calling certbot in test mode against a local pebble instance
* Add a command to start pebble/boulder
* Use explicit start
* Add execution permission to acme_server
* Add a docstring to certbot_test function
* Change executable name
* Increase sleep to 3600s
* Implement a context manager to handle the acme server
* Add certbot_test workspace in .gitignore
* Add documentation
* Remove one function in context, split logic of certbot_test towards capturing non capturing
* Use an explicit an properly configured ACMEServer as handler.
* Add doc. Put constants.
2019-06-12 20:19:23 -04:00
|
|
|
force_renew=False)
|
2019-04-15 18:09:57 -04:00
|
|
|
|
|
|
|
|
assert_cert_count_for_lineage(context.config_dir, certname, 2)
|
|
|
|
|
assert_hook_execution(context.hook_probe, 'deploy')
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
def test_renew_with_changed_private_key_complexity(context):
|
|
|
|
|
"""Test proper renew with updated private key complexity."""
|
|
|
|
|
certname = context.get_domain('renew')
|
|
|
|
|
context.certbot(['-d', certname, '--rsa-key-size', '4096'])
|
|
|
|
|
|
|
|
|
|
key1 = join(context.config_dir, 'archive', certname, 'privkey1.pem')
|
|
|
|
|
assert os.stat(key1).st_size > 3000 # 4096 bits keys takes more than 3000 bytes
|
|
|
|
|
assert_cert_count_for_lineage(context.config_dir, certname, 1)
|
|
|
|
|
|
|
|
|
|
context.certbot(['renew'])
|
|
|
|
|
|
|
|
|
|
assert_cert_count_for_lineage(context.config_dir, certname, 2)
|
|
|
|
|
key2 = join(context.config_dir, 'archive', certname, 'privkey2.pem')
|
|
|
|
|
assert os.stat(key2).st_size > 3000
|
|
|
|
|
|
|
|
|
|
context.certbot(['renew', '--rsa-key-size', '2048'])
|
|
|
|
|
|
|
|
|
|
assert_cert_count_for_lineage(context.config_dir, certname, 3)
|
|
|
|
|
key3 = join(context.config_dir, 'archive', certname, 'privkey3.pem')
|
|
|
|
|
assert os.stat(key3).st_size < 1800 # 2048 bits keys takes less than 1800 bytes
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
def test_renew_ignoring_directory_hooks(context):
|
|
|
|
|
"""Test hooks are ignored during renewal with relevant CLI flag."""
|
|
|
|
|
certname = context.get_domain('renew')
|
|
|
|
|
context.certbot(['-d', certname])
|
|
|
|
|
|
|
|
|
|
assert_cert_count_for_lineage(context.config_dir, certname, 1)
|
|
|
|
|
|
|
|
|
|
misc.generate_test_file_hooks(context.config_dir, context.hook_probe)
|
|
|
|
|
context.certbot(['renew', '--no-directory-hooks'])
|
|
|
|
|
|
|
|
|
|
assert_cert_count_for_lineage(context.config_dir, certname, 2)
|
|
|
|
|
with pytest.raises(AssertionError):
|
|
|
|
|
assert_hook_execution(context.hook_probe, 'deploy')
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
def test_renew_empty_hook_scripts(context):
|
|
|
|
|
"""Test proper renew with empty hook scripts."""
|
|
|
|
|
certname = context.get_domain('renew')
|
|
|
|
|
context.certbot(['-d', certname])
|
|
|
|
|
|
|
|
|
|
assert_cert_count_for_lineage(context.config_dir, certname, 1)
|
|
|
|
|
|
|
|
|
|
misc.generate_test_file_hooks(context.config_dir, context.hook_probe)
|
|
|
|
|
for hook_dir in misc.list_renewal_hooks_dirs(context.config_dir):
|
|
|
|
|
shutil.rmtree(hook_dir)
|
|
|
|
|
os.makedirs(join(hook_dir, 'dir'))
|
|
|
|
|
open(join(hook_dir, 'file'), 'w').close()
|
|
|
|
|
context.certbot(['renew'])
|
|
|
|
|
|
|
|
|
|
assert_cert_count_for_lineage(context.config_dir, certname, 2)
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
def test_renew_hook_override(context):
|
|
|
|
|
"""Test correct hook override on renew."""
|
|
|
|
|
certname = context.get_domain('override')
|
|
|
|
|
context.certbot([
|
|
|
|
|
'certonly', '-d', certname,
|
|
|
|
|
'--preferred-challenges', 'http-01',
|
2019-08-06 18:02:16 -04:00
|
|
|
'--pre-hook', misc.echo('pre', context.hook_probe),
|
|
|
|
|
'--post-hook', misc.echo('post', context.hook_probe),
|
|
|
|
|
'--deploy-hook', misc.echo('deploy', context.hook_probe),
|
2019-04-15 18:09:57 -04:00
|
|
|
])
|
|
|
|
|
|
|
|
|
|
assert_hook_execution(context.hook_probe, 'pre')
|
|
|
|
|
assert_hook_execution(context.hook_probe, 'post')
|
|
|
|
|
assert_hook_execution(context.hook_probe, 'deploy')
|
|
|
|
|
|
|
|
|
|
# Now we override all previous hooks during next renew.
|
|
|
|
|
open(context.hook_probe, 'w').close()
|
|
|
|
|
context.certbot([
|
|
|
|
|
'renew', '--cert-name', certname,
|
2019-08-06 18:02:16 -04:00
|
|
|
'--pre-hook', misc.echo('pre_override', context.hook_probe),
|
|
|
|
|
'--post-hook', misc.echo('post_override', context.hook_probe),
|
|
|
|
|
'--deploy-hook', misc.echo('deploy_override', context.hook_probe),
|
2019-04-15 18:09:57 -04:00
|
|
|
])
|
|
|
|
|
|
2019-08-06 18:02:16 -04:00
|
|
|
assert_hook_execution(context.hook_probe, 'pre_override')
|
|
|
|
|
assert_hook_execution(context.hook_probe, 'post_override')
|
|
|
|
|
assert_hook_execution(context.hook_probe, 'deploy_override')
|
2019-04-15 18:09:57 -04:00
|
|
|
with pytest.raises(AssertionError):
|
|
|
|
|
assert_hook_execution(context.hook_probe, 'pre')
|
|
|
|
|
with pytest.raises(AssertionError):
|
|
|
|
|
assert_hook_execution(context.hook_probe, 'post')
|
|
|
|
|
with pytest.raises(AssertionError):
|
|
|
|
|
assert_hook_execution(context.hook_probe, 'deploy')
|
|
|
|
|
|
|
|
|
|
# Expect that this renew will reuse new hooks registered in the previous renew.
|
|
|
|
|
open(context.hook_probe, 'w').close()
|
|
|
|
|
context.certbot(['renew', '--cert-name', certname])
|
|
|
|
|
|
2019-08-06 18:02:16 -04:00
|
|
|
assert_hook_execution(context.hook_probe, 'pre_override')
|
|
|
|
|
assert_hook_execution(context.hook_probe, 'post_override')
|
|
|
|
|
assert_hook_execution(context.hook_probe, 'deploy_override')
|
|
|
|
|
|
2019-04-15 19:18:24 -04:00
|
|
|
|
|
|
|
|
def test_invalid_domain_with_dns_challenge(context):
|
|
|
|
|
"""Test certificate issuance failure with DNS-01 challenge."""
|
|
|
|
|
# Manual dns auth hooks from misc are designed to fail if the domain contains 'fail-*'.
|
|
|
|
|
domains = ','.join([context.get_domain('dns1'), context.get_domain('fail-dns1')])
|
|
|
|
|
context.certbot([
|
|
|
|
|
'-a', 'manual', '-d', domains,
|
|
|
|
|
'--allow-subset-of-names',
|
|
|
|
|
'--preferred-challenges', 'dns',
|
|
|
|
|
'--manual-auth-hook', context.manual_dns_auth_hook,
|
|
|
|
|
'--manual-cleanup-hook', context.manual_dns_cleanup_hook
|
|
|
|
|
])
|
|
|
|
|
|
|
|
|
|
output = context.certbot(['certificates'])
|
|
|
|
|
|
|
|
|
|
assert context.get_domain('fail-dns1') not in output
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
def test_reuse_key(context):
|
|
|
|
|
"""Test various scenarios where a key is reused."""
|
|
|
|
|
certname = context.get_domain('reusekey')
|
|
|
|
|
context.certbot(['--domains', certname, '--reuse-key'])
|
|
|
|
|
context.certbot(['renew', '--cert-name', certname])
|
|
|
|
|
|
|
|
|
|
with open(join(context.config_dir, 'archive/{0}/privkey1.pem').format(certname), 'r') as file:
|
|
|
|
|
privkey1 = file.read()
|
|
|
|
|
with open(join(context.config_dir, 'archive/{0}/privkey2.pem').format(certname), 'r') as file:
|
|
|
|
|
privkey2 = file.read()
|
|
|
|
|
assert privkey1 == privkey2
|
|
|
|
|
|
|
|
|
|
context.certbot(['--cert-name', certname, '--domains', certname, '--force-renewal'])
|
|
|
|
|
|
|
|
|
|
with open(join(context.config_dir, 'archive/{0}/privkey3.pem').format(certname), 'r') as file:
|
|
|
|
|
privkey3 = file.read()
|
|
|
|
|
assert privkey2 != privkey3
|
|
|
|
|
|
|
|
|
|
with open(join(context.config_dir, 'archive/{0}/cert1.pem').format(certname), 'r') as file:
|
|
|
|
|
cert1 = file.read()
|
|
|
|
|
with open(join(context.config_dir, 'archive/{0}/cert2.pem').format(certname), 'r') as file:
|
|
|
|
|
cert2 = file.read()
|
|
|
|
|
with open(join(context.config_dir, 'archive/{0}/cert3.pem').format(certname), 'r') as file:
|
|
|
|
|
cert3 = file.read()
|
|
|
|
|
|
2019-04-15 19:42:06 -04:00
|
|
|
assert len({cert1, cert2, cert3}) == 3
|
2019-04-17 18:24:39 -04:00
|
|
|
|
|
|
|
|
|
|
|
|
|
def test_ecdsa(context):
|
|
|
|
|
"""Test certificate issuance with ECDSA key."""
|
|
|
|
|
key_path = join(context.workspace, 'privkey-p384.pem')
|
|
|
|
|
csr_path = join(context.workspace, 'csr-p384.der')
|
|
|
|
|
cert_path = join(context.workspace, 'cert-p384.pem')
|
|
|
|
|
chain_path = join(context.workspace, 'chain-p384.pem')
|
|
|
|
|
|
|
|
|
|
misc.generate_csr([context.get_domain('ecdsa')], key_path, csr_path, key_type=misc.ECDSA_KEY_TYPE)
|
|
|
|
|
context.certbot(['auth', '--csr', csr_path, '--cert-path', cert_path, '--chain-path', chain_path])
|
|
|
|
|
|
|
|
|
|
certificate = misc.read_certificate(cert_path)
|
|
|
|
|
assert 'ASN1 OID: secp384r1' in certificate
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
def test_ocsp_must_staple(context):
|
|
|
|
|
"""Test that OCSP Must-Staple is correctly set in the generated certificate."""
|
|
|
|
|
if context.acme_server == 'pebble':
|
|
|
|
|
pytest.skip('Pebble does not support OCSP Must-Staple.')
|
|
|
|
|
|
|
|
|
|
certname = context.get_domain('must-staple')
|
|
|
|
|
context.certbot(['auth', '--must-staple', '--domains', certname])
|
|
|
|
|
|
|
|
|
|
certificate = misc.read_certificate(join(context.config_dir,
|
|
|
|
|
'live/{0}/cert.pem').format(certname))
|
|
|
|
|
assert 'status_request' in certificate or '1.3.6.1.5.5.7.1.24' in certificate
|
|
|
|
|
|
|
|
|
|
|
2019-04-15 19:42:06 -04:00
|
|
|
def test_revoke_simple(context):
|
|
|
|
|
"""Test various scenarios that revokes a certificate."""
|
|
|
|
|
# Default action after revoke is to delete the certificate.
|
|
|
|
|
certname = context.get_domain()
|
|
|
|
|
cert_path = join(context.config_dir, 'live', certname, 'cert.pem')
|
|
|
|
|
context.certbot(['-d', certname])
|
|
|
|
|
context.certbot(['revoke', '--cert-path', cert_path, '--delete-after-revoke'])
|
|
|
|
|
|
|
|
|
|
assert not exists(cert_path)
|
|
|
|
|
|
|
|
|
|
# Check default deletion is overridden.
|
|
|
|
|
certname = context.get_domain('le1')
|
|
|
|
|
cert_path = join(context.config_dir, 'live', certname, 'cert.pem')
|
|
|
|
|
context.certbot(['-d', certname])
|
|
|
|
|
context.certbot(['revoke', '--cert-path', cert_path, '--no-delete-after-revoke'])
|
|
|
|
|
|
|
|
|
|
assert exists(cert_path)
|
|
|
|
|
|
|
|
|
|
context.certbot(['delete', '--cert-name', certname])
|
|
|
|
|
|
|
|
|
|
assert not exists(join(context.config_dir, 'archive', certname))
|
|
|
|
|
assert not exists(join(context.config_dir, 'live', certname))
|
|
|
|
|
assert not exists(join(context.config_dir, 'renewal', '{0}.conf'.format(certname)))
|
|
|
|
|
|
|
|
|
|
certname = context.get_domain('le2')
|
|
|
|
|
key_path = join(context.config_dir, 'live', certname, 'privkey.pem')
|
|
|
|
|
cert_path = join(context.config_dir, 'live', certname, 'cert.pem')
|
|
|
|
|
context.certbot(['-d', certname])
|
|
|
|
|
context.certbot(['revoke', '--cert-path', cert_path, '--key-path', key_path])
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
def test_revoke_and_unregister(context):
|
|
|
|
|
"""Test revoke with a reason then unregister."""
|
|
|
|
|
cert1 = context.get_domain('le1')
|
|
|
|
|
cert2 = context.get_domain('le2')
|
|
|
|
|
cert3 = context.get_domain('le3')
|
|
|
|
|
|
|
|
|
|
cert_path1 = join(context.config_dir, 'live', cert1, 'cert.pem')
|
|
|
|
|
key_path2 = join(context.config_dir, 'live', cert2, 'privkey.pem')
|
|
|
|
|
cert_path2 = join(context.config_dir, 'live', cert2, 'cert.pem')
|
|
|
|
|
|
|
|
|
|
context.certbot(['-d', cert1])
|
|
|
|
|
context.certbot(['-d', cert2])
|
|
|
|
|
context.certbot(['-d', cert3])
|
|
|
|
|
|
|
|
|
|
context.certbot(['revoke', '--cert-path', cert_path1,
|
|
|
|
|
'--reason', 'cessationOfOperation'])
|
|
|
|
|
context.certbot(['revoke', '--cert-path', cert_path2, '--key-path', key_path2,
|
|
|
|
|
'--reason', 'keyCompromise'])
|
|
|
|
|
|
|
|
|
|
context.certbot(['unregister'])
|
|
|
|
|
|
|
|
|
|
output = context.certbot(['certificates'])
|
|
|
|
|
|
|
|
|
|
assert cert1 not in output
|
|
|
|
|
assert cert2 not in output
|
|
|
|
|
assert cert3 in output
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
def test_revoke_mutual_exclusive_flags(context):
|
|
|
|
|
"""Test --cert-path and --cert-name cannot be used during revoke."""
|
|
|
|
|
cert = context.get_domain('le1')
|
|
|
|
|
context.certbot(['-d', cert])
|
|
|
|
|
with pytest.raises(subprocess.CalledProcessError) as error:
|
|
|
|
|
context.certbot([
|
|
|
|
|
'revoke', '--cert-name', cert,
|
|
|
|
|
'--cert-path', join(context.config_dir, 'live', cert, 'fullchain.pem')
|
|
|
|
|
])
|
|
|
|
|
assert 'Exactly one of --cert-path or --cert-name must be specified' in error.out
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
def test_revoke_multiple_lineages(context):
|
|
|
|
|
"""Test revoke does not delete certs if multiple lineages share the same dir."""
|
|
|
|
|
cert1 = context.get_domain('le1')
|
|
|
|
|
context.certbot(['-d', cert1])
|
|
|
|
|
|
|
|
|
|
assert os.path.isfile(join(context.config_dir, 'renewal', '{0}.conf'.format(cert1)))
|
|
|
|
|
|
|
|
|
|
cert2 = context.get_domain('le2')
|
|
|
|
|
context.certbot(['-d', cert2])
|
|
|
|
|
|
|
|
|
|
# Copy over renewal configuration of cert1 into renewal configuration of cert2.
|
|
|
|
|
with open(join(context.config_dir, 'renewal', '{0}.conf'.format(cert2)), 'r') as file:
|
|
|
|
|
data = file.read()
|
|
|
|
|
|
|
|
|
|
data = re.sub('archive_dir = .*\n',
|
2019-08-06 18:02:16 -04:00
|
|
|
'archive_dir = {0}\n'.format(join(context.config_dir, 'archive', cert1).replace('\\', '\\\\')),
|
2019-04-15 19:42:06 -04:00
|
|
|
data)
|
|
|
|
|
|
|
|
|
|
with open(join(context.config_dir, 'renewal', '{0}.conf'.format(cert2)), 'w') as file:
|
|
|
|
|
file.write(data)
|
|
|
|
|
|
|
|
|
|
output = context.certbot([
|
|
|
|
|
'revoke', '--cert-path', join(context.config_dir, 'live', cert1, 'cert.pem')
|
|
|
|
|
])
|
|
|
|
|
|
|
|
|
|
assert 'Not deleting revoked certs due to overlapping archive dirs' in output
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
def test_wildcard_certificates(context):
|
|
|
|
|
"""Test wildcard certificate issuance."""
|
|
|
|
|
if context.acme_server == 'boulder-v1':
|
|
|
|
|
pytest.skip('Wildcard certificates are not supported on ACME v1')
|
|
|
|
|
|
|
|
|
|
certname = context.get_domain('wild')
|
|
|
|
|
|
|
|
|
|
context.certbot([
|
|
|
|
|
'-a', 'manual', '-d', '*.{0},{0}'.format(certname),
|
|
|
|
|
'--preferred-challenge', 'dns',
|
|
|
|
|
'--manual-auth-hook', context.manual_dns_auth_hook,
|
|
|
|
|
'--manual-cleanup-hook', context.manual_dns_cleanup_hook
|
|
|
|
|
])
|
|
|
|
|
|
|
|
|
|
assert exists(join(context.config_dir, 'live', certname, 'fullchain.pem'))
|
2019-04-15 20:39:38 -04:00
|
|
|
|
|
|
|
|
|
|
|
|
|
def test_ocsp_status_stale(context):
|
|
|
|
|
"""Test retrieval of OCSP statuses for staled config"""
|
|
|
|
|
sample_data_path = misc.load_sample_data_path(context.workspace)
|
|
|
|
|
output = context.certbot(['certificates', '--config-dir', sample_data_path])
|
|
|
|
|
|
|
|
|
|
assert output.count('TEST_CERT') == 2, ('Did not find two test certs as expected ({0})'
|
|
|
|
|
.format(output.count('TEST_CERT')))
|
|
|
|
|
assert output.count('EXPIRED') == 2, ('Did not find two expired certs as expected ({0})'
|
|
|
|
|
.format(output.count('EXPIRED')))
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
def test_ocsp_status_live(context):
|
|
|
|
|
"""Test retrieval of OCSP statuses for live config"""
|
2019-08-02 14:46:12 -04:00
|
|
|
cert = context.get_domain('ocsp-check')
|
2019-04-15 20:39:38 -04:00
|
|
|
|
|
|
|
|
# OSCP 1: Check live certificate OCSP status (VALID)
|
|
|
|
|
context.certbot(['--domains', cert])
|
|
|
|
|
output = context.certbot(['certificates'])
|
|
|
|
|
|
|
|
|
|
assert output.count('VALID') == 1, 'Expected {0} to be VALID'.format(cert)
|
|
|
|
|
assert output.count('EXPIRED') == 0, 'Did not expect {0} to be EXPIRED'.format(cert)
|
|
|
|
|
|
|
|
|
|
# OSCP 2: Check live certificate OCSP status (REVOKED)
|
|
|
|
|
context.certbot(['revoke', '--cert-name', cert, '--no-delete-after-revoke'])
|
2019-05-14 16:56:32 -04:00
|
|
|
# Sometimes in oldest tests (using openssl binary and not cryptography), the OCSP status is
|
|
|
|
|
# not seen immediately by Certbot as invalid. Waiting few seconds solves this transient issue.
|
|
|
|
|
time.sleep(5)
|
2019-04-15 20:39:38 -04:00
|
|
|
output = context.certbot(['certificates'])
|
|
|
|
|
|
|
|
|
|
assert output.count('INVALID') == 1, 'Expected {0} to be INVALID'.format(cert)
|
|
|
|
|
assert output.count('REVOKED') == 1, 'Expected {0} to be REVOKED'.format(cert)
|