summary refs log tree commit diff
diff options
context:
space:
mode:
authorLucas Savva <lucas@m1cr0man.com>2021-11-26 21:39:06 +0000
committerLucas Savva <lucas@m1cr0man.com>2021-12-26 16:44:07 +0000
commita88d846b91dc3c75b825187998debd8e08c7794b (patch)
treedffd38914c860df742e6017bc035dd2b0c56eb07
parent52c1d49b71dfb3cae6dedcc04ace8da79399e09d (diff)
downloadnixpkgs-a88d846b91dc3c75b825187998debd8e08c7794b.tar
nixpkgs-a88d846b91dc3c75b825187998debd8e08c7794b.tar.gz
nixpkgs-a88d846b91dc3c75b825187998debd8e08c7794b.tar.bz2
nixpkgs-a88d846b91dc3c75b825187998debd8e08c7794b.tar.lz
nixpkgs-a88d846b91dc3c75b825187998debd8e08c7794b.tar.xz
nixpkgs-a88d846b91dc3c75b825187998debd8e08c7794b.tar.zst
nixpkgs-a88d846b91dc3c75b825187998debd8e08c7794b.zip
nixos/acme: Remove selfsignedDeps from finished targets
selfsignedDeps is already appended to the after and wants
of a cert's renewal service, making these redundant.

You can see this if you run the following command:
systemctl list-dependencies --all --reverse acme-selfsigned-mydomain.com.service
-rw-r--r--nixos/modules/security/acme.nix4
1 files changed, 2 insertions, 2 deletions
diff --git a/nixos/modules/security/acme.nix b/nixos/modules/security/acme.nix
index dcadbce68aa..dcea137b52e 100644
--- a/nixos/modules/security/acme.nix
+++ b/nixos/modules/security/acme.nix
@@ -842,8 +842,8 @@ in {
         # Create some targets which can be depended on to be "active" after cert renewals
         finishedTargets = mapAttrs' (cert: conf: nameValuePair "acme-finished-${cert}" {
           wantedBy = [ "default.target" ];
-          requires = [ "acme-${cert}.service" ] ++ conf.selfsignedDeps;
-          after = [ "acme-${cert}.service" ] ++ conf.selfsignedDeps;
+          requires = [ "acme-${cert}.service" ];
+          after = [ "acme-${cert}.service" ];
         }) certConfigs;
 
         # Create targets to limit the number of simultaneous account creations