Make the log directory configurable

Author: gotthardp

doc/Installation.md

@@ -51,10 +51,12 @@ You can run the server by:
 bin/lorawan-server
 ```
 
-By default the database directory `Mnesia.lorawan@*` and the `log` files are
-stored in the `lorawan-server` directory you created. If you want to store the
-run-time information to another directory, set the `LORAWAN_HOME` environment
-variable.
+By default the database directory `Mnesia.lorawan@*` is stored in
+the `lorawan-server` directory you created. If you want to store the run-time
+information to another directory, set the `LORAWAN_HOME` environment variable.
+
+The `log` files are by default stored in the same directory. If you want to
+store the log files elsewhere, set the `LORAWAN_LOG_ROOT` variable.
 
 The lorawan-server can be started in background as a daemon.
 On Linux systems with systemd you should:
@@ -63,7 +65,8 @@ On Linux systems with systemd you should:
  * Create a dedicated user by `useradd --home-dir /var/lib/lorawan-server --create-home lorawan`
  * Start the server by `systemctl start lorawan-server`
 
-This will put the database and the logs into `/var/lib/lorawan-server`.
+This will put the database into `/var/lib/lorawan-server` and server logs into
+`/var/log/lorawan-server`.
 
 ### Using the Binary Release on Windows
 

lorawan_server.config

@@ -37,9 +37,10 @@
     % {error_logger_hwm, undefined},
     {handlers, [
         % {lager_console_backend, [{level, debug}]},
-        {lager_file_backend, [{file, "log/debug.log"}, {level, debug}]},
-        {lager_file_backend, [{file, "log/error.log"}, {level, error}]}
-    ]}
+        {lager_file_backend, [{file, "debug.log"}, {level, debug}]},
+        {lager_file_backend, [{file, "error.log"}, {level, error}]}
+    ]},
+    {crash_log, "crash.log"}
 ]},
 {sasl, [
     {errlog_type, error}

scripts/lorawan-server

@@ -17,8 +17,13 @@ elif [ -L "$__possible_sys".orig ]; then
     SYS_CONFIG="$__possible_sys"
 fi
 
+if [ ! -z $LORAWAN_LOG_ROOT ]; then
+    ERL_ARGS="-lager log_root \"$LORAWAN_LOG_ROOT\""
+else
+    ERL_ARGS="-lager log_root \"log\""
+fi
 if [ ! -z $GOOGLE_MAPS_KEY ]; then
     sed -i "s/GoogleMapsKey = \"[-a-zA-Z0-9]*\"/GoogleMapsKey = \"$GOOGLE_MAPS_KEY\"/g" $ROOT_DIR/lib/lorawan_server-*/priv/admin/admin-config.js
 fi
 
-cd $LORAWAN_HOME && erl -noinput +Bd -sname lorawan -pa $ROOT_DIR/lib/*/ebin -s lorawan_app -config $SYS_CONFIG $@
+cd $LORAWAN_HOME && erl -noinput +Bd -sname lorawan -pa $ROOT_DIR/lib/*/ebin -s lorawan_app $ERL_ARGS -config $SYS_CONFIG $@

scripts/lorawan-server.bat

@@ -4,7 +4,7 @@ setlocal EnableDelayedExpansion
 for /D %%A in ("%PROGRAMFILES%\erl*" "%PROGRAMFILES(x86)%\erl*") do (
     for /D %%B in ("%%A\erts*") do (
         for /D %%C in ("%%B\bin\erl.exe") do (
-            set "ERLDIR=%%C"
+            set "ERL_DIR=%%C"
         )
     )
 )
@@ -18,4 +18,5 @@ for /D %%A in ("%ROOT_DIR%\lib\*") do (
     set FILES=!FILES! "%%A\ebin"
 )
 
-cd %ROOT_DIR% && %ERLDIR% -noinput +Bd -sname lorawan -pa !FILES! -s lorawan_app -config releases/{{release_version}}/sys.config
+set "ERL_ARGS=-lager log_root "log""
+cd %ROOT_DIR% && %ERL_DIR% -noinput +Bd -sname lorawan -pa !FILES! -s lorawan_app %ERL_ARGS% -config releases/{{release_version}}/sys.config

scripts/lorawan-server.service

@@ -5,6 +5,7 @@ After=network.target
 [Service]
 Type=simple
 Environment=LORAWAN_HOME=/var/lib/lorawan-server
+Environment=LORAWAN_LOG_ROOT=/var/log/lorawan-server
 ExecStart=/usr/lib/lorawan-server/bin/lorawan-server
 ExecStop=/usr/lib/lorawan-server/bin/lorawanctl stop
 User=lorawan