Feature #296

Can't use cronolog with server.errorlog

Added by Anonymous almost 5 years ago. Updated over 1 year ago.

Status:Fixed Start:
Priority:Low Due date:
Assigned to:- % Done:

100%
Category:core
Target version:1.4.23
Missing in 1.5.x:

Description

It appears that cronolog can only be used with the accesslog.filename.

When server.errorlog is configured to pipe to cronolog, the following occurs:

Starting lighttpd.
2005-09-28 23:02:22: (log.c.60) opening errorlog '|/usr/local/sbin/cronolog /home/anthony/logs/domain.net-error_log.%Y%m%d' failed: No such file or directory
2005-09-28 23:02:22: (server.c.711) opening errorlog failed, dying

#> lighttpd -v
lighttpd-1.4.3 - a light and fast webserver
Build-Date: Sep 28 2005 01:27:03

This behavior is not consistent with accesslog.filename and should be either corrected or documented. Apache, for instance, allows one to use cronolog for both sets of logs. That is a more consistent configuration.

lighttpd-1.4.x-piperrorlog.patch (4.6 KB) carenas, 12/17/2006 11:44 pm

lighttpd-piperrorlog.patch - patch for trunk that adds a new errorlog type "pipe" and implements it (3.6 KB) carenas, 12/18/2006 02:36 am

log-cronolog2.diff - updated original patch for new version of lighty -- rkitover (2.5 KB) Anonymous, 11/05/2007 05:20 am

log-cronolog3.diff - updated patch for 1.5 (trunk) (2.5 KB) rkitover, 01/20/2009 12:09 am

Associated revisions

Revision 2466
Added by stbuehler over 1 year ago

Add support for pipe logging for server.errorlog (fixes #296)

History

Updated by Anonymous over 4 years ago

  • Status changed from New to Assigned

I have written a patch for this (well, blatantly ripped off from mod_accesslog.c):

http://blog.moria.org.uk/downloads/log-cronolog.diff

file, syslog and stderr logging all seem to work still, and I have this working to cronolog okay here. Works independently of whether mod_accesslog is also to cronolog (or equiv.).

-- cph

Updated by Anonymous over 2 years ago

patch: ** Only garbage was found in the patch input.

Updated by Anonymous over 2 years ago

-- Lfe

Updated by Anonymous about 2 years ago

Any news?

Updated by stbuehler almost 2 years ago

  • Target version changed from 1.4.20 to 1.4.21

Updated by akrus almost 2 years ago

Bump! Any news?

Updated by rkitover over 1 year ago

Updated by icy over 1 year ago

  • Target version changed from 1.4.21 to 1.4.22
  • Patch available set to No

Updated by stbuehler over 1 year ago

  • Target version changed from 1.4.22 to 1.4.23

Updated by stbuehler over 1 year ago

  • Assigned to deleted (stbuehler)
  • Target version changed from 1.4.23 to 1.4.x

I don't see any reason to add this upstream - just use stderr and pipe it yourself.

Updated by stbuehler over 1 year ago

  • Status changed from Assigned to Fixed
  • % Done changed from 0 to 100

Applied in changeset r2466.

Updated by stbuehler over 1 year ago

  • Target version changed from 1.4.x to 1.4.23

I hope our gentoo users are happy now :)

Also available in: Atom