search

strongloop / modern-syslog

modern-syslog
Other
49 stars 19 forks source link

Fix ignored facilities #9

Closed rmg closed 9 years ago

rmg commented 9 years ago

I believe this to be the root of #8. The problem isn't specific to FreeBSD, but the fact that Haraka was using syslog.open() to specify a log facility by number (syslog.LOG_MAIL instead of 'LOG_MAIL').

It's a pretty trivial change, so we should be able to publish a patch release shortly.

@sam-github PTAL /cc @Dexus @msimerson

Connect to #8

msimerson commented 9 years ago

:+1: thanks greatly for your effort in tracking that down.

msimerson commented 9 years ago

I just tested that patch against my production Haraka and it does indeed solve the problem.

Dexus commented 9 years ago

:+1: now it also runtastic :100: on docker image.

Thanks Ryan. Can only agree with Matt.

sam-github commented 9 years ago

LGTM, can you publish after merge?

rmg commented 9 years ago

Released modern-syslog@1.1.1