Revert "nixos/jicofo: use ExecStart instead of script"

This reverts commit d3a26a5ecd81e3aa1618679e05759083c9b5a9af.
Using ServiceConfig.ExecStart instead of script lead to the content not
being executed in a shell anymore, which broke the secrets being read
from a file and passed as a command line parameter.

1 files changed, 10 insertions, 9 deletions

diff --git a/nixos/modules/services/networking/jicofo.nix b/nixos/modules/services/networking/jicofo.nix
index 2142d8f7591..8c492600944 100644
--- a/nixos/modules/services/networking/jicofo.nix
+++ b/nixos/modules/services/networking/jicofo.nix
@@ -105,16 +105,17 @@ in
       ];
       environment.JAVA_SYS_PROPS = concatStringsSep " " (mapAttrsToList (k: v: "${k}=${toString v}") jicofoProps);
 
+      script = ''
+        ${pkgs.jicofo}/bin/jicofo \
+          --host=${cfg.xmppHost} \
+          --domain=${if cfg.xmppDomain == null then cfg.xmppHost else cfg.xmppDomain} \
+          --secret=$(cat ${cfg.componentPasswordFile}) \
+          --user_name=${cfg.userName} \
+          --user_domain=${cfg.userDomain} \
+          --user_password=$(cat ${cfg.userPasswordFile})
+      '';
+
       serviceConfig = {
-        ExecStart = ''
-          ${pkgs.jicofo}/bin/jicofo \
-            --host=${cfg.xmppHost} \
-            --domain=${if cfg.xmppDomain == null then cfg.xmppHost else cfg.xmppDomain} \
-            --secret=$(cat ${cfg.componentPasswordFile}) \
-            --user_name=${cfg.userName} \
-            --user_domain=${cfg.userDomain} \
-            --user_password=$(cat ${cfg.userPasswordFile})
-        '';
         Type = "exec";
 
         DynamicUser = true;