Re: Syslog Facility Patch

Larry Rosenman writes:

> Here is a patch for allowing the syslog facility to be set.

I think you want to check for invalid values, like "LOCAL37.5". The last
field in the ConfigNamesString structure (see below) allows you to set a
"parse hook", that is, a function that parses the input value and returns
true if it is okay. At least this is how it is supposed to work, I've
never actually tried it. :) Or maybe just make it an integer option?

> ***************
> *** 303,308 ****
> --- 304,312 ----
>
> {"unix_socket_group", PGC_POSTMASTER, &Unix_socket_group,
> "", NULL},
> + #ifdef ENABLE_SYSLOG
> + {"syslog_facility", PGC_SIGHUP, &Syslog_facility, "LOCAL0", NULL},
> + #endif
>
> {NULL, 0, NULL, NULL, NULL}
> };
>
>

--
Peter Eisentraut peter_e(at)gmx(dot)net http://yi.org/peter-e/