Feature #296

Can't use cronolog with server.errorlog

Added by Anonymous about 9 years ago. Updated over 5 years ago.

Status:FixedStart date:
Priority:LowDue date:
Assignee:-% 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 Magnifier (4.64 KB) carenas, 2006-12-17 23:44

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

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

log-cronolog3.diff Magnifier - updated patch for 1.5 (trunk) (2.45 KB) rkitover, 2009-01-20 00:09

Associated revisions

Revision 2466
Added by stbuehler over 5 years ago

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

History

#1 Updated by Anonymous almost 9 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

#2 Updated by Anonymous almost 7 years ago

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

#3 Updated by Anonymous over 6 years ago

-- Lfe

#4 Updated by Anonymous about 6 years ago

Any news?

#5 Updated by stbuehler about 6 years ago

  • Target version changed from 1.4.20 to 1.4.21

#6 Updated by akrus about 6 years ago

Bump! Any news?

#7 Updated by rkitover almost 6 years ago

#8 Updated by icy over 5 years ago

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

#9 Updated by stbuehler over 5 years ago

  • Target version changed from 1.4.22 to 1.4.23

#10 Updated by stbuehler over 5 years ago

  • Assignee 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.

#11 Updated by stbuehler over 5 years ago

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

Applied in changeset r2466.

#12 Updated by stbuehler over 5 years ago

  • Target version changed from 1.4.x to 1.4.23

I hope our gentoo users are happy now :)

Also available in: Atom