Merge pull request #1137 from offlinehacker/nixos/redis/fix_user_syslog

nixos/redis: user set uid, log to syslog by default

nixos/modules/misc/ids.nix

@@ -102,6 +102,7 @@
       tcpcryptd = 93; # tcpcryptd uses a hard-coded uid. We patch it in Nixpkgs to match this choice.
       zope2 = 94;
       firebird = 95;
+      redis = 96;
 
       # When adding a uid, make sure it doesn't match an existing gid.
 

nixos/modules/services/databases/redis.nix

@@ -14,6 +14,7 @@ let
     ${condOption "unixsocket" cfg.unixSocket}
     loglevel ${cfg.logLevel}
     logfile ${cfg.logfile}
+    syslog-enabled ${redisBool cfg.syslog}
     databases ${toString cfg.databases}
     ${concatMapStrings (d: "save ${toString (builtins.elemAt d 0)} ${toString (builtins.elemAt d 1)}\n") cfg.save}
     dbfilename ${cfg.dbFilename}
@@ -82,12 +83,18 @@ in
       };
 
       logfile = mkOption {
-        default = "stdout";
+        default = "/dev/null";
         description = "Specify the log file name. Also 'stdout' can be used to force Redis to log on the standard output.";
         example = "/var/log/redis.log";
         type = with types; string;
       };
 
+      syslog = mkOption {
+        default = true;
+        description = "Enable logging to the system logger.";
+        type = with types; bool;
+      };
+
       databases = mkOption {
         default = 16;
         description = "Set the number of databases.";
@@ -177,8 +184,9 @@ in
 
   config = mkIf config.services.redis.enable {
 
-    users.extraUsers = singleton
+    users.extraUsers.redis =
       { name = cfg.user;
+        uid = config.ids.uids.redis;
         description = "Redis database user";
       };