From 7cbafeaacd9676ffd8e1d2ed12051409bc524ddb Mon Sep 17 00:00:00 2001 From: Olli-Antti Kivilahti 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