From d21f72abf4050c40bdf4a60afe0f49a8b94f44b6 Mon Sep 17 00:00:00 2001
From: glyph <mail@apophenic.net>
Date: Wed, 17 Jun 2026 18:00:44 +0200
Subject: [PATCH] nixos/sabnzbd: offload config format to
 pkgs.formats.configobj

---
 .../services/networking/sabnzbd/default.nix   | 73 ++++---------------
 1 file changed, 13 insertions(+), 60 deletions(-)

diff --git a/nixos/modules/services/networking/sabnzbd/default.nix b/nixos/modules/services/networking/sabnzbd/default.nix
index d4a894faf96f..7791423c5943 100644
--- a/nixos/modules/services/networking/sabnzbd/default.nix
+++ b/nixos/modules/services/networking/sabnzbd/default.nix
@@ -29,71 +29,24 @@ let
 
   cfg = config.services.sabnzbd;
 
+  # Sabnzbd expects 0/1 instead of true/false
+  fixupSettings = lib.mapAttrsRecursive (
+    path: value:
+    if value == true then
+      1
+    else if value == false then
+      0
+    else
+      value
+  );
+
   mandatoryGlobalSettings = {
     "__version__" = 19;
     "__encoding__" = "utf-8";
   };
-  allSettings = mandatoryGlobalSettings // cfg.settings;
+  allSettings = fixupSettings (mandatoryGlobalSettings // cfg.settings);
 
-  # sabnzbd uses configobj type inis, which support
-  # nested sections specified by increasing numbers
-  # of square brackets (but not toml style dotted paths)
-  configObjAtom = types.oneOf [
-    types.str
-    types.int
-    types.bool
-  ];
-
-  configObjValue =
-    let
-      valueType =
-        types.oneOf [
-          types.str
-          types.int
-          types.bool
-          (types.listOf configObjAtom)
-          (types.attrsOf valueType)
-        ]
-        // {
-          description = "ConfigObj type";
-        };
-    in
-    valueType;
-
-  configObjIni =
-    { }:
-    let
-      extractAtoms = lib.filterAttrs (k: v: v != null && !lib.isAttrs v);
-      extractSections = lib.filterAttrs (k: v: lib.isAttrs v);
-      mkValueString = (
-        v:
-        if true == v then
-          "1"
-        else if false == v then
-          "0"
-        else
-          mkValueStringDefault { } v
-      );
-      mkKeyValue = mkKeyValueDefault { inherit mkValueString; } "=";
-      mkSection = (
-        depth: attrs:
-        let
-          sections = extractSections attrs;
-          sectionHeadingLeft = lib.concatStrings (lib.replicate (depth + 1) "[");
-          sectionHeadingRight = lib.concatStrings (lib.replicate (depth + 1) "]");
-          mkSectionHeading =
-            name: "${sectionHeadingLeft}${lib.escape [ "[" "]" ] name}${sectionHeadingRight}";
-          mkSubsection = name: attrs: (mkSectionHeading name) + "\n" + (mkSection (depth + 1) attrs) + "\n";
-        in
-        toKeyValue { inherit mkKeyValue; } (extractAtoms attrs)
-        + "\n"
-        + lib.concatStrings (lib.mapAttrsToList mkSubsection sections)
-      );
-    in
-    {
-      type = types.attrsOf configObjValue;
-      generate = name: attrs: pkgs.writeText name (mkSection 0 attrs);
-    };
+  configObjIni = pkgs.formats.configobj;
 
   publicSettingsIni =
     if cfg.configFile != null then