Make RPC password a secret

modules/bitcoind.nix

@@ -28,7 +28,6 @@ let
     ${cfg.extraConfig}
   '';
   cmdlineOptions = concatMapStringsSep " " (arg: "'${arg}'") [
-    "-conf=${configFile}"
     "-datadir=${cfg.dataDir}"
     "-pid=${pidFile}"
   ];
@@ -179,14 +178,16 @@ in {
     environment.systemPackages = [ cfg.package ];
     systemd.services.bitcoind = {
       description = "Bitcoin daemon";
-      after = [ "network.target" ];
+      requires = [ "bitcoin-rpcpassword-key.service" ];
+      after = [ "network.target" "bitcoin-rpcpassword-key.service" ];
       wantedBy = [ "multi-user.target" ];
       preStart = ''
         if ! test -e ${cfg.dataDir}; then
           mkdir -m 0770 -p '${cfg.dataDir}'
           chown '${cfg.user}:${cfg.group}' '${cfg.dataDir}'
         fi
-        ln -sf '${configFile}' '${cfg.dataDir}/bitcoin.conf'
+        cp '${configFile}' '${cfg.dataDir}/bitcoin.conf'
+        echo "rpcpassword=$(cat /secrets/bitcoin-rpcpassword)" >> '${cfg.dataDir}/bitcoin.conf'
       '';
       serviceConfig = {
         Type = "simple";
@@ -212,6 +213,7 @@ in {
       name = cfg.user;
       #uid  = config.ids.uids.bitcoin;
       group = cfg.group;
+      extraGroups = [ "keys" ];
       description = "Bitcoin daemon user";
       home = cfg.dataDir;
     };

modules/clightning.nix

@@ -9,7 +9,6 @@ let
     autolisten=false
     network=bitcoin
     bitcoin-rpcuser=${cfg.bitcoin-rpcuser}
-    bitcoin-rpcpassword=${cfg.bitcoin-rpcpassword}
   '';
 in {
   options.services.clightning = {
@@ -33,12 +32,6 @@ in {
         Bitcoin RPC user
       '';
     };
-    bitcoin-rpcpassword = mkOption {
-      type = types.string;
-      description = ''
-        Bitcoin RPC password
-      '';
-    };
   };
 
   config = mkIf cfg.enable {
@@ -46,17 +39,21 @@ in {
         {
           description = "clightning User";
           createHome  = true;
+          extraGroups = [ "bitcoinrpc" "keys" ];
           inherit home;
       };
       systemd.services.clightning =
         { description = "Run clightningd";
-          path  = [ pkgs.clightning pkgs.bitcoin ];
+          path  = [ pkgs.bash pkgs.clightning pkgs.bitcoin ];
           wantedBy = [ "multi-user.target" ];
           requires = [ "bitcoind.service" ];
           after = [ "bitcoind.service" ];
           preStart = ''
             mkdir -p ${home}/.lightning
-            ln -sf ${configFile} ${home}/.lightning/config
+            rm -f ${home}/.lightning/config
+            cp ${configFile} ${home}/.lightning/config
+            chmod +w ${home}/.lightning/config
+            echo "bitcoin-rpcpassword=$(cat /secrets/bitcoin-rpcpassword)" >> '${home}/.lightning/config'
             '';
           serviceConfig =
             {

modules/nixbitcoin.nix

@@ -4,7 +4,6 @@ with lib;
 
 let
   cfg = config.services.nixbitcoin;
-  secrets = import ../load-secrets.nix;
 in {
   imports =
     [
@@ -24,6 +23,8 @@ in {
   };
 
   config = mkIf cfg.enable {
+    users.groups.bitcoinrpc = {};
+
     # Tor
     services.tor.enable = true;
     services.tor.client.enable = true;
@@ -40,7 +41,6 @@ in {
     services.bitcoind.proxy = config.services.tor.client.socksListenAddress;
     services.bitcoind.port = 8333;
     services.bitcoind.rpcuser = "bitcoinrpc";
-    services.bitcoind.rpcpassword = secrets.bitcoinrpcpassword;
     services.bitcoind.extraConfig = ''
       assumevalid=0000000000000000000726d186d6298b5054b9a5c49639752294b322a305d240
       addnode=ecoc5q34tmbq54wl.onion
@@ -51,7 +51,6 @@ in {
     # clightning
     services.clightning.enable = true;
     services.clightning.bitcoin-rpcuser = config.services.bitcoind.rpcuser;
-    services.clightning.bitcoin-rpcpassword = config.services.bitcoind.rpcpassword;
 
     # nodeinfo
     systemd.services.nodeinfo = {