Attachment 139811 Details for Bug 18205 – Bug 18205: [ALTERNATE] Use Koha::Logger for Mojolicious API logger

[patch] Bug 18205: [ALTERNATE] Use Koha::Logger for Mojolicious API logger

Bug-18205-ALTERNATE-Use-KohaLogger-for-Mojolicious.patch (text/plain), 1.17 KB, created by David Cook on 2022-08-26 00:04:09 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: David Cook

Created: 2022-08-26 00:04:09 UTC

Size: 1.17 KB

patch

obsolete

>From 98401fc685cad62c2b73122754f3eab618f3252d Mon Sep 17 00:00:00 2001
>From: David Cook <dcook@prosentient.com.au>
>Date: Thu, 25 Aug 2022 23:59:42 +0000
>Subject: [PATCH] Bug 18205: [ALTERNATE] Use Koha::Logger for Mojolicious API
> logger
>
>This patch adds Koha::Logger as the Mojolicious logging object
>for the API Mojolicious app.
>
>Test plan:
>0. Apply the patch and koha-plack --restart kohadev
>1. Add $c->app->log->debug('DEBUG') and
>$c->app->log->warn('WARN') to a API controller like
>Koha::REST::V1::Libraries
>2. koha-plack --restart kohadev
>3. Go to the API route tied to that API controller method
>4. Look at /var/log/koha/kohadev/api-error.log
>5. Note that 'WARN' appears but 'DEBUG' does not
>---
> Koha/REST/V1.pm | 3 +++
> 1 file changed, 3 insertions(+)
>
>diff --git a/Koha/REST/V1.pm b/Koha/REST/V1.pm
>index 62d326e0fd..79d73bbb87 100644
>--- a/Koha/REST/V1.pm
>+++ b/Koha/REST/V1.pm
>@@ -43,6 +43,9 @@ Overloaded Mojolicious->startup method. It is called at application startup.
> sub startup {
>     my $self = shift;
> 
>+    my $logger = Koha::Logger->get({ interface => 'api' });
>+    $self->log($logger);
>+
>     $self->hook(
>         before_dispatch => sub {
>             my $c = shift;
>-- 
>2.25.1

Actions: View | Diff | Splinter Review

Attachments on bug 18205: 60848 | 60849 | 60850 | 69226 | 69227 | 69228 | 69232 | 85711 | 85712 | 85713 | 85714 | 87537 | 87538 | 87539 | 87540 | 87541 | 87542 | 87545 | 139811