aboutsummaryrefslogtreecommitdiff
path: root/syslogd
diff options
context:
space:
mode:
authorDavid Oberhollenzer <david.oberhollenzer@tele2.at>2018-08-19 10:16:31 +0200
committerDavid Oberhollenzer <david.oberhollenzer@tele2.at>2018-08-20 07:44:15 +0200
commit61bc850984db2f31970865a724b120543e679dec (patch)
tree7a047e2b69bf9692e9376596ce69b2add8be9b56 /syslogd
parentf51dca08787ebb6fb7a37ae2777a5a8a90e9a657 (diff)
usyslogd: merge log files for the same service
If a service provides an identifier string, write all messages for that service itno a file named after that identifier instread of splitting it up by facility ID in a sub directory. This is supposed to cleanup and simplify the clutter created in /var/log. Signed-off-by: David Oberhollenzer <david.oberhollenzer@tele2.at>
Diffstat (limited to 'syslogd')
-rw-r--r--syslogd/logfile.c90
1 files changed, 47 insertions, 43 deletions
diff --git a/syslogd/logfile.c b/syslogd/logfile.c
index 0347ab7..6240905 100644
--- a/syslogd/logfile.c
+++ b/syslogd/logfile.c
@@ -45,30 +45,30 @@ static const enum_map_t levels[] = {
};
static const enum_map_t facilities[] = {
- { "kernel.log", 0 },
- { "user.log", 1 },
- { "mail.log", 2 },
- { "daemon.log", 3 },
- { "auth.log", 4 },
- { "syslog.log", 5 },
- { "lpr.log", 6 },
- { "news.log", 7 },
- { "uucp.log", 8 },
- { "clock.log", 9 },
- { "authpriv.log", 10 },
- { "ftp.log", 11 },
- { "ntp.log", 12 },
- { "audit.log", 13 },
- { "alert.log", 14 },
- { "cron.log", 15 },
- { "local0.log", 16 },
- { "local1.log", 17 },
- { "local2.log", 18 },
- { "local3.log", 19 },
- { "local4.log", 20 },
- { "local5.log", 21 },
- { "local6.log", 22 },
- { "local7.log", 23 },
+ { "kernel", 0 },
+ { "user", 1 },
+ { "mail", 2 },
+ { "daemon", 3 },
+ { "auth", 4 },
+ { "syslog", 5 },
+ { "lpr", 6 },
+ { "news", 7 },
+ { "uucp", 8 },
+ { "clock", 9 },
+ { "authpriv", 10 },
+ { "ftp", 11 },
+ { "ntp", 12 },
+ { "audit", 13 },
+ { "alert", 14 },
+ { "cron", 15 },
+ { "local0", 16 },
+ { "local1", 17 },
+ { "local2", 18 },
+ { "local3", 19 },
+ { "local4", 20 },
+ { "local5", 21 },
+ { "local6", 22 },
+ { "local7", 23 },
{ NULL, 0 },
};
@@ -135,7 +135,7 @@ static logfile_t *logfile_create(const char *filename)
static int logfile_write(logfile_t *file, const syslog_msg_t *msg)
{
- const char *lvl_str;
+ const char *lvl_str, *fac_name;
char timebuf[32];
struct tm tm;
int ret;
@@ -150,8 +150,17 @@ static int logfile_write(logfile_t *file, const syslog_msg_t *msg)
gmtime_r(&msg->timestamp, &tm);
strftime(timebuf, sizeof(timebuf), "%FT%T", &tm);
- ret = dprintf(file->fd, "[%s][%s][%u] %s", timebuf, lvl_str, msg->pid,
- msg->message);
+ if (msg->ident != NULL) {
+ fac_name = enum_to_name(facilities, msg->facility);
+ if (fac_name == NULL)
+ return -1;
+
+ ret = dprintf(file->fd, "[%s][%s][%s][%u] %s", timebuf,
+ fac_name, lvl_str, msg->pid, msg->message);
+ } else {
+ ret = dprintf(file->fd, "[%s][%s][%u] %s", timebuf, lvl_str,
+ msg->pid, msg->message);
+ }
fsync(file->fd);
@@ -229,35 +238,30 @@ static void file_backend_cleanup(log_backend_t *backend)
static int file_backend_write(log_backend_t *backend, const syslog_msg_t *msg)
{
log_backend_file_t *log = (log_backend_file_t *)backend;
- const char *fac_name;
+ const char *ident;
char *filename;
logfile_t *f;
size_t len;
- fac_name = enum_to_name(facilities, msg->facility);
- if (fac_name == NULL)
- return -1;
-
- if (msg->ident) {
- len = strlen(msg->ident) + 1 + strlen(fac_name) + 1;
- filename = alloca(len);
- sprintf(filename, "%s/%s", msg->ident, fac_name);
+ if (msg->ident != NULL) {
+ ident = msg->ident;
} else {
- filename = (char *)fac_name;
+ ident = enum_to_name(facilities, msg->facility);
+ if (ident == NULL)
+ return -1;
}
+ len = strlen(ident) + strlen(".log") + 1;
+ filename = alloca(len);
+ strcpy(filename, ident);
+ strcat(filename, ".log");
+
for (f = log->list; f != NULL; f = f->next) {
if (strcmp(filename, f->filename) == 0)
break;
}
if (f == NULL) {
- if (msg->ident != NULL && mkdir(msg->ident, 0750) != 0 &&
- errno != EEXIST) {
- perror(msg->ident);
- return -1;
- }
-
f = logfile_create(filename);
if (f == NULL)
return -1;