Merge pull request #888 from Samasaur1/fix-sudo-extraconfig

security.sudo.extraConfig: fix default behavior
LnL7 · Feb 27, 2024 · 6c06334 · 6c06334
2 parents bbde06b + 4488831
commit 6c06334
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/modules/security/sudo.nix b/modules/security/sudo.nix
@@ -12,15 +12,19 @@ in
 
   options = {
     security.sudo.extraConfig = mkOption {
-      type = types.lines;
-      default = "";
+      type = types.nullOr types.lines;
+      default = null;
       description = mdDoc ''
         Extra configuration text appended to {file}`sudoers`.
       '';
     };
   };
 
   config = {
-    environment.etc."sudoers.d/10-nix-darwin-extra-config".text = lib.mkIf (cfg.extraConfig != "") cfg.extraConfig;
+    environment.etc = {
+      "sudoers.d/10-nix-darwin-extra-config" = mkIf (cfg.extraConfig != null) {
+        text = cfg.extraConfig;
+      };
+    };
   };
 }