search

augensalat / mojolicious-plugin-accesslog

Easily generate access logs for Mojolicious applications
4 stars 4 forks source link

the plugin produces a warning in the logs whenever Mojolicious starts #9

Closed akarelas closed 3 years ago

akarelas commented 8 years ago 
==> api.err.log <==
[Sat Nov 21 22:41:50 2015] [warn] Mojolicious::Plugin::AccessLog::VERSION = 0.010

I think this is useless, and shouldn't be there since I didn't do anything wrong to get warned about.

akarelas commented 8 years ago

And it's pretty annoying when using Morbo (which restarts often)

jberger commented 7 years ago

👍 it is getting on the nerves of our ops staff. Even if it is useful, which I'm not convinced about, it certainly isn't warn level.

christopherraa commented 7 years ago

@jberger @akarelas I have a fork of this module in which this is fixed, as well as the addition of a placeholder (%x) that extracts the outermost client in the X-Forwarded-For chain so you can log it easily. Thought I should mention it.

Grinnz commented 7 years ago

Agreed with above comments

karenetheridge commented 3 years ago

This looks like an easy fix - any chance it can be merged?

augensalat commented 3 years ago

Fixed with release 0.010001.

karenetheridge commented 3 years ago

thanks very much!