Project

General

Profile

Actions

Refactor #25896

closed

Syslog logging pattern should not include time or request id

Added by Lukas Zapletal about 5 years ago. Updated about 5 years ago.

Status:
Closed
Priority:
Normal
Category:
Logging
Target version:
-
Difficulty:
trivial
Triaged:
Yes
Fixed in Releases:
Found in Releases:

Description

The sys_pattern was created to differentiate between file appender and syslog appender:

:pattern: "%d [%.1l|%.3c|%.8X{request}] m\n" 
:sys_pattern: "[
.1l|%.3c|%.8X{request}] %m\n"

It does not make sense to have the same defaults for both, syslog should really be just %m (message itself), all the rest can be added in syslog configuration when needed.


Related issues 1 (1 open0 closed)

Related to Foreman - Tracker #22569: Unified and structured loggingAssignedLukas Zapletal

Actions
Actions #1

Updated by Lukas Zapletal about 5 years ago

Actions #2

Updated by The Foreman Bot about 5 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/6428 added
Actions #3

Updated by Tomer Brisker about 5 years ago

  • Fixed in Releases 1.22.0 added
Actions #4

Updated by Lukas Zapletal about 5 years ago

  • Status changed from Ready For Testing to Closed
Actions

Also available in: Atom PDF