Bugzilla Bug 222373

I've written a patch which allows the use of syslog in nginx.See
http://article.gmane.org/gmane.comp.web.nginx.english/5057

Reproducible: Always

Created an attachment (id=153307) [details]
nginx syslog support

Created an attachment (id=153311) [details]
made use of LOG_CRIT + LOG_NOTICE

new patch with different log facilities, instead of logging everything to
LOG_CRIT

Created an attachment (id=153345) [details]
used format function for syslog() to allow %s, %d, etc in the string

would be cool if Igor could at least make a response there :/

(In reply to comment #4)
> would be cool if Igor could at least make a response there :/
> 

Author of Nginx (Igor) does not like logging to syslog in nginx - see mail
archive

I'm trying to write an ebuild that includes this patch with the syslog USE
flag, but the patch isn't fully working (about half of the hunks fail) with
nginx 0.8.4

Any chance there's an updated patch?

Created an attachment (id=197160) [details]
add syslog support to ebuild

This patch adds the syslog nginx patch and compiles nginx with syslog support
when the syslog USE flag is present.

I'll take a look to make it working against the latest 0.7 and 0.8 branch

Created an attachment (id=197180) [details]
syslog patch for nginx 0.8.4

I updated the patch to work with nginx-0.8.4

I've tested this with the ebuild patch attached to this ticket, and it works
with my nginx configs, but the patch might be messy (when I tried compiling by
hand, it had some warnings), so if someone could check it out, that'd be great.

I'm a bit biased toward third party patching in Nginx. For instance - if this
goes through, do we also accept mod_wsgi patches or perhaps passenger? This
small and easy to use web server can easily grow into something more complex.

At this time I don't want to increase the maintenance load by pulling in more
stuff from outside the distribution, so closing as WONTFIX.