[PATCH] Syslog Logging Not Working on FreeBSD

[david@…]

FreeBSD's syslog doesn't listen on /dev/log. It uses /var/run/log. Attached is a patch that allows you to specify the path to the UNIX socket instead of hardcoding it. It basically uses the logfile parameter, if it's None use /dev/log otherwise use it.

This bug affects all versions of Trac since about 0.7 it looks like.

[david@…]

Patch for 0.10-stable and trunk. Overrides handling of UNIX socket path via logfile parameter.

[david@…]

It's also possible to just remove the explicit path, and everything Just Works(tm) (at least on Python 2.5.2).

[Christian Boos]

My preference goes to attachment:syslog-removepath.patch​.

Moving to 0.13 so that we don't have to bother with Python 2.3.

Should be verified for 2.4 as well, and on other platforms if needed (MacOS X?)

[george@…]

Yes, this works for OS 10.6 with the built-in Python 2.6.

[Remy Blank]

According to the Python documentation:

• Passing no argument to SysLogHandler connects to the syslog via UDP port 514 on localhost.
• Specifying a string argument to SysLogHandler is only documented since 2.5, but is actually supported earlier (2.3 is the oldest I checked), and connects to a UNIX socket.

Considering this, we could support both variants, which comes closer to the first patch:

• Add a new parameter logdest.
• If logdest starts with UDP:, we pass the remaining string as a hostname and log via UDP to that host (which could be localhost). This would also allow logging remotely.
• Otherwise, logdest is passed as a string and we log to a UNIX socket.

Re-using logfile for this purpose is probably a bad idea, as the default value is trac.log.

[Remy Blank]

Oops, wrong field :)

[Ryan J Ollos]

#11216 closed as a duplicate, additionally requesting the ability to set the syslog facility. ticket:11216:logging_config.patch​ adds a log_config option which points to a dedicated logging configuration file.

[Sebastian Klemke <packet@…>]

Updated logging config patch. Adds the path, basename, project log format substitutions.

[Sebastian Klemke <packet@…>]

Logging config ini file to log to syslog on FreeBSD

[Sebastian Klemke <packet@…>]

Patch to the admin panel to add log_config support.

[Ryan J Ollos]

The patch looks pretty good. The biggest thing that will delay getting this committed is the lack of unit tests. If some unit tests were added this could be included in milestone:1.2.

[Ryan J Ollos]

See also #11874. The proposed changes in that ticket would be a good follow-on to this ticket.

[Ryan J Ollos]

Minor refactorings in [14333:14336].

Revised changes in log:rjollos.git:t7820_logging_config. The revised changes avoid API changes that were present in attachment:logging_config.patch​. Some of those changes have been deferred to #11874 (comment:10:ticket:11874).

I'll do more testing and add unit tests.

[Ryan J Ollos]

TODO Add documentation to TracLogging.

[Jun Omae]

At least, we should use disable_existing_loggers=False for the logging.config.fileConfig(). I guess that call of the fileConfig() would stop loggers of other environments with multiple projects.

See:

• ​https://docs.python.org/2/library/logging.config.html#logging.config.fileConfig
• ​https://docs.python.org/3/howto/logging.html#configuring-logging