Ticket #1484850 (closed Patches: fixed)

Opened 9 months ago

Last modified 3 months ago

Option to use syslog instead of log file

Reported by: rjhughes Owned by:
Priority: 5 Milestone: 0.2-beta
Component: PHP backend Version: 0.1-stable
Severity: normal Keywords:
Cc:

Description

Hi. I'm continuing my quest to make standard services available in roundcube.

Here, I added an option to allow syslog logging, rather than simply writing to a file. Syslog can do anything, it's great. I left the 'file' driver as the default, though.

Attachments

syslog.diff (5.6 kB) - added by rjhughes 9 months ago.
Patch against 1186 to allow the use of syslog, but continue to use file-based logging if you don't specify.

Change History

follow-up: ↓ 2   Changed 9 months ago by thomasb

  • component changed from Client to PHP backend
  • milestone changed from 0.1.1 to 0.2-beta

I would rather put it the other way round:

if ($CONFIG['log_driver'] == 'syslog') { ... }
else { /* current code */ }

In this case the config option 'log_driver' remains optional.

in reply to: ↑ 1   Changed 9 months ago by rjhughes

Okay, the latest attachment does it that way -- if you omit the log_driver config statement entirely, it will behave as if you'd said 'file'.

Changed 9 months ago by rjhughes

Patch against 1186 to allow the use of syslog, but continue to use file-based logging if you don't specify.

  Changed 3 months ago by alec

  • status changed from new to closed
  • resolution set to fixed

Added in r1683.

Note: See TracTickets for help on using tickets.