From 7cbafeaacd9676ffd8e1d2ed12051409bc524ddb Mon Sep 17 00:00:00 2001
From: Olli-Antti Kivilahti <olli-antti.kivilahti@jns.fi>
Date: Thu, 11 Jun 2015 17:48:54 +0300
Subject: [PATCH] Bug 13799 - Set up logging and configuration file reading
 for Mojolicious.

Use environmental values to control some aspects of Mojolicious:
This way we can have different settings for different servers running Mojolicious.

%%%% Configuration file %%%%

$ENV{MOJO_CONFIG} should be set in the system service (init) starting Mojolicious, eg:
export MOJO_CONFIG=/home/koha/kohaclone/api/v1/hypnotoad.conf

This configuration file read by the Mojolicious::Plugin::Config
http://mojolicio.us/perldoc/Mojolicious/Plugin/Config

%%%%% Logging %%%%%

>NOTE!!
>There is a "feature" in Mojo::Server disabling STDOUT and STDERR, because such errors are not-suited-for-prod~
>This modification in Mojo::Server disables this and preserves the STD* handles for forked server threads
>in Mojo::Server::daemonize(), comment out the following lines
>
># Close filehandles
>#  open STDOUT, '>/dev/null';
>#  open STDERR, '>&STDOUT';

Log to a filename configured in an environemnt variable $ENV{MOJO_LOGFILE} using loglevel $ENV{MOJO_LOGLEVEL}.
Defaults to '/tmp/koha-api.log' and loglevel of 'error'
Examples:
export MOJO_LOGFILE=/home/koha/koha-dev/var/log/kohaapi.mojo.log
export MOJO_LOGLEVEL=debug

Logging is done by Mojo::Log
http://www.mojolicio.us/perldoc/Mojo/Log
---
 Koha/REST/V1.pm |   62 +++++++++++++++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 62 insertions(+)

diff --git a/Koha/REST/V1.pm b/Koha/REST/V1.pm
index 3becf12..089461b 100644
--- a/Koha/REST/V1.pm
+++ b/Koha/REST/V1.pm
@@ -2,10 +2,72 @@ package Koha::REST::V1;
 
 use Modern::Perl;
 use Mojo::Base 'Mojolicious';
+use Mojo::Log;
+
+=head startup
+
+Starts the Mojolicious application using whatever server is configured.
+
+Use environmental values to control some aspects of Mojolicious:
+This way we can have different settings for different servers running Mojolicious.
+
+=head2 Configuration file
+
+$ENV{MOJO_CONFIG} should be set in the system service (init) starting Mojolicious, eg:
+export MOJO_CONFIG=/home/koha/kohaclone/api/v1/hypnotoad.conf
+
+This configuration file read by the Mojolicious::Plugin::Config
+http://mojolicio.us/perldoc/Mojolicious/Plugin/Config
+
+=head2 Logging
+
+    #NOTE!!
+    #There is a "feature" in Mojo::Server disabling STDOUT and STDERR, because such errors are not-suited-for-production?!?
+    #This modification in Mojo::Server disables this and preserves the STD* handles for forked server threads
+    #in Mojo::Server::daemonize(), comment out the following lines
+    #
+    #  # Close filehandles
+    #  open STDOUT, '>/dev/null';
+    #  open STDERR, '>&STDOUT';
+
+Log to a filename configured in an environment variable $ENV{MOJO_LOGFILES} using loglevel $ENV{MOJO_LOGLEVEL}.
+Actually you get 3 logfiles on $ENV{MOJO_LOGFILES}.
+.log for Mojo::Log
+.stdout for STDOUT
+.stderr for STDERR
+
+Defaults to STDERR and loglevel of 'error'
+Examples:
+export MOJO_LOGFILES=/home/koha/koha-dev/var/log/kohaapi.mojo
+export MOJO_LOGLEVEL=debug
+
+Logging is done by Mojo::Log
+http://www.mojolicio.us/perldoc/Mojo/Log
+
+=cut
 
 sub startup {
     my $self = shift;
 
+    #Log to a filename with loglevel configured in environment variables
+    if ($ENV{MOJO_LOGFILES}) {
+        $self->app->log( Mojo::Log->new( path => $ENV{MOJO_LOGFILES}.'.log', level => ($ENV{MOJO_LOGLEVEL} || 'error') ) );
+        open(STDOUT,'>>',$ENV{MOJO_LOGFILES}.'.stdout') or die __PACKAGE__."::startup():> Couldn't open the STDOUT logfile for appending.\n".$!;
+        open(STDERR,'>>',$ENV{MOJO_LOGFILES}.'.stderr') or die __PACKAGE__."::startup():> Couldn't open the STDERR logfile for appending.\n".$!;
+    }
+    else {
+        $self->app->log(); #Default to STDERR
+        print __PACKAGE__."::startup():> No logfile given, defaulting to STDERR. Define your logfile and loglevel to the MOJO_LOGFILE and MOJO_LOGLEVEL environmental variables.\n";
+    }
+
+    #Enable the config-plugin. Loads the config file from $ENV{MOJO_CONFIG} by default.
+    if ($ENV{MOJO_CONFIG}) {
+        $self->plugin('Config');
+    }
+    else {
+        print __PACKAGE__."::startup():> No config-file loaded. Define your config-file to the MOJO_CONFIG environmental variable.\n";
+    }
+
     my $route = $self->routes->under->to(
         cb => sub {
             my $c = shift;
-- 
1.7.9.5