Skip Menu |

This queue is for tickets about the Mojo-Log-Syslog CPAN distribution.

Report information
The Basics
Id: 130647
Status: resolved
Priority: 0/
Queue: Mojo-Log-Syslog

People
Owner: SGRAY [...] cpan.org
Requestors: sch [...] chaos.dk
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: 1.02



Subject: Mojo::Log::Syslog broken in modern (8.25) Mojolicious
Date: Fri, 4 Oct 2019 18:09:18 +0200
To: bug-Mojo-Log-Syslog [...] rt.cpan.org
From: Søren Schrøder <sch [...] chaos.dk>
Changes in Mojo::Log breaks Mojo::Log::Syslog 1.01 Works in 8.17, broken in 8.25 it seems Mojo::Log was changed in 8.24 8.24 2019-09-11 - Added EXPERIMENTAL context method to Mojo::Log. -- Søren Schrøder. Obey Gravity - It's the law !
Hi Søren, Show quoted text
> Works in 8.17, broken in 8.25
Can you give more detail please? What's the actual behavior with 8.25?
Subject: Re: [rt.cpan.org #130647] Mojo::Log::Syslog broken in modern (8.25) Mojolicious
Date: Mon, 7 Oct 2019 10:18:15 +0200
To: bug-Mojo-Log-Syslog [...] rt.cpan.org
From: Søren Schrøder <sch [...] chaos.dk>
Using this small example: Show quoted text
-------------start----------- use Mojolicious::Lite; use Data::Dumper; use Mojo::Log::Syslog; use Mojo::Log; my $log = new Mojo::Log::Syslog(facility => 'local0', ident => "oltprov", level => 'info'); app->log($log); app->log->info("LITE is starting"); any '/' => sub { my $c=shift; app->log->info("GOT /"); $c->render(text => "hello ".time()); }; app->config(hypnotoad => {listen => ['http://*:3001'], pid_file => 'lite.pid'}); app->log->level('debug'); app->start; --------- end ----------------- run under morbo: morbo -l http://*:80 foo.pl under 8.07 and 8.21 and it works fine under 8.25 I get this error on a "GET /" event Mojo::Reactor::EV: I/O watcher failed: unrecognized arguments at /usr/local/share/perl/5.24.1/Mojolicious/Plugin/DefaultHelpers.pm line 175. CORE Perl (v5.24.1, linux) Mojolicious (8.25, Supervillain) OPTIONAL Cpanel::JSON::XS 4.09+ (n/a) EV 4.0+ (4.22) IO::Socket::Socks 0.64+ (0.73) IO::Socket::SSL 2.009+ (2.044) Net::DNS::Native 0.15+ (n/a) Role::Tiny 2.000001+ (2.000005) On Fri, Oct 4, 2019 at 8:00 PM Sergey Poznyakoff via RT < bug-Mojo-Log-Syslog@rt.cpan.org> wrote:
> <URL: https://rt.cpan.org/Ticket/Display.html?id=130647 > > > Hi Søren, >
> > Works in 8.17, broken in 8.25
> > Can you give more detail please? What's the actual behavior with 8.25? >
-- Søren Schrøder. Obey Gravity - It's the law !
Thanks a lot for the perfect testcase. Please, try this patch: http://git.gnu.org.ua/cgit/mojosyslog.git/patch/?id=0a916b8b229d6ffb6d4a1b821221ec16a3b5187f Let me know if it works for you.
Subject: Re: [rt.cpan.org #130647] Mojo::Log::Syslog broken in modern (8.25) Mojolicious
Date: Thu, 10 Oct 2019 14:09:54 +0200
To: bug-Mojo-Log-Syslog [...] rt.cpan.org
From: Søren Schrøder <sch [...] chaos.dk>
I have tested your patch in my apps, and it works fine in 8.25 now (and keeps working in 8.21 and found an old 8.09 where it also works). Thanks for your swift work ! On Wed, Oct 9, 2019 at 2:04 PM Sergey Poznyakoff via RT < bug-Mojo-Log-Syslog@rt.cpan.org> wrote: Show quoted text
> <URL: https://rt.cpan.org/Ticket/Display.html?id=130647 > > > Thanks a lot for the perfect testcase. Please, try this patch: > > > http://git.gnu.org.ua/cgit/mojosyslog.git/patch/?id=0a916b8b229d6ffb6d4a1b821221ec16a3b5187f > > Let me know if it works for you. > > > > >
-- Søren Schrøder. Obey Gravity - It's the law !
Thank you. I'm closing the issue.