Merge pull request #99912 from m1cr0man/ocspfix

nixos/acme: Fix ocspMustStaple option and add test

Changed files
+43 -5
nixos
modules
security
tests
common
acme
server
+4 -1
nixos/modules/security/acme.nix
···
"--email" data.email
"--key-type" data.keyType
] ++ protocolOpts
-
++ optionals data.ocspMustStaple [ "--must-staple" ]
++ optionals (acmeServer != null) [ "--server" acmeServer ]
++ concatMap (name: [ "-d" name ]) extraDomains
++ data.extraLegoFlags;
runOpts = escapeShellArgs (
commonOpts
++ [ "run" ]
++ data.extraLegoRunFlags
);
renewOpts = escapeShellArgs (
commonOpts
++ [ "renew" "--reuse-key" ]
++ data.extraLegoRenewFlags
);
···
"--email" data.email
"--key-type" data.keyType
] ++ protocolOpts
++ optionals (acmeServer != null) [ "--server" acmeServer ]
++ concatMap (name: [ "-d" name ]) extraDomains
++ data.extraLegoFlags;
+
# Although --must-staple is common to both modes, it is not declared as a
+
# mode-agnostic argument in lego and thus must come after the mode.
runOpts = escapeShellArgs (
commonOpts
++ [ "run" ]
+
++ optionals data.ocspMustStaple [ "--must-staple" ]
++ data.extraLegoRunFlags
);
renewOpts = escapeShellArgs (
commonOpts
++ [ "renew" "--reuse-key" ]
+
++ optionals data.ocspMustStaple [ "--must-staple" ]
++ data.extraLegoRenewFlags
);
+38 -3
nixos/tests/acme.nix
···
};
};
# Test using Apache HTTPD
specialisation.httpd-aliases.configuration = { pkgs, config, lib, ... }: {
services.nginx.enable = lib.mkForce false;
···
testScript = {nodes, ...}:
let
newServerSystem = nodes.webserver.config.system.build.toplevel;
switchToNewServer = "${newServerSystem}/bin/switch-to-configuration test";
in
···
return check_connection_key_bits(node, domain, bits, retries - 1)
client.start()
dnsserver.start()
···
client.wait_for_unit("default.target")
client.succeed(
-
'curl --data \'{"host": "acme.test", "addresses": ["${nodes.acme.config.networking.primaryIPAddress}"]}\' http://${dnsServerIP nodes}:8055/add-a'
)
acme.start()
···
acme.wait_for_unit("default.target")
acme.wait_for_unit("pebble.service")
-
client.succeed("curl https://acme.test:15000/roots/0 > /tmp/ca.crt")
-
client.succeed("curl https://acme.test:15000/intermediate-keys/0 >> /tmp/ca.crt")
with subtest("Can request certificate with HTTPS-01 challenge"):
webserver.wait_for_unit("acme-finished-a.example.test.target")
···
webserver.wait_for_unit("acme-finished-a.example.test.target")
check_connection_key_bits(client, "a.example.test", "384")
webserver.succeed("grep testing /var/lib/acme/a.example.test/test")
with subtest("Can request certificate with HTTPS-01 when nginx startup is delayed"):
switch_to(webserver, "slow-startup")
···
};
};
+
# Test OCSP Stapling
+
specialisation.ocsp-stapling.configuration = { pkgs, ... }: {
+
security.acme.certs."a.example.test" = {
+
ocspMustStaple = true;
+
};
+
services.nginx.virtualHosts."a.example.com" = {
+
extraConfig = ''
+
ssl_stapling on;
+
ssl_stapling_verify on;
+
'';
+
};
+
};
+
# Test using Apache HTTPD
specialisation.httpd-aliases.configuration = { pkgs, config, lib, ... }: {
services.nginx.enable = lib.mkForce false;
···
testScript = {nodes, ...}:
let
+
caDomain = nodes.acme.config.test-support.acme.caDomain;
newServerSystem = nodes.webserver.config.system.build.toplevel;
switchToNewServer = "${newServerSystem}/bin/switch-to-configuration test";
in
···
return check_connection_key_bits(node, domain, bits, retries - 1)
+
def check_stapling(node, domain, retries=3):
+
assert retries >= 0
+
+
# Pebble doesn't provide a full OCSP responder, so just check the URL
+
result = node.succeed(
+
"openssl s_client -CAfile /tmp/ca.crt"
+
f" -servername {domain} -connect {domain}:443 < /dev/null"
+
" | openssl x509 -noout -ocsp_uri"
+
)
+
print("OCSP Responder URL:", result)
+
+
if "${caDomain}:4002" not in result.lower():
+
time.sleep(1)
+
return check_stapling(node, domain, retries - 1)
+
+
client.start()
dnsserver.start()
···
client.wait_for_unit("default.target")
client.succeed(
+
'curl --data \'{"host": "${caDomain}", "addresses": ["${nodes.acme.config.networking.primaryIPAddress}"]}\' http://${dnsServerIP nodes}:8055/add-a'
)
acme.start()
···
acme.wait_for_unit("default.target")
acme.wait_for_unit("pebble.service")
+
client.succeed("curl https://${caDomain}:15000/roots/0 > /tmp/ca.crt")
+
client.succeed("curl https://${caDomain}:15000/intermediate-keys/0 >> /tmp/ca.crt")
with subtest("Can request certificate with HTTPS-01 challenge"):
webserver.wait_for_unit("acme-finished-a.example.test.target")
···
webserver.wait_for_unit("acme-finished-a.example.test.target")
check_connection_key_bits(client, "a.example.test", "384")
webserver.succeed("grep testing /var/lib/acme/a.example.test/test")
+
+
with subtest("Correctly implements OCSP stapling"):
+
switch_to(webserver, "ocsp-stapling")
+
webserver.wait_for_unit("acme-finished-a.example.test.target")
+
check_stapling(client, "a.example.test")
with subtest("Can request certificate with HTTPS-01 when nginx startup is delayed"):
switch_to(webserver, "slow-startup")
+1 -1
nixos/tests/common/acme/server/default.nix
···
privateKey = testCerts.${domain}.key;
httpPort = 80;
tlsPort = 443;
-
ocspResponderURL = "http://0.0.0.0:4002";
strict = true;
};
···
privateKey = testCerts.${domain}.key;
httpPort = 80;
tlsPort = 443;
+
ocspResponderURL = "http://${domain}:4002";
strict = true;
};