Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

nginx_syslog_patch in upstream? #6

Closed
rngadam opened this issue Jan 15, 2013 · 7 comments
Closed

nginx_syslog_patch in upstream? #6

rngadam opened this issue Jan 15, 2013 · 7 comments

Comments

@rngadam
Copy link

rngadam commented Jan 15, 2013

I was wondering; will this patch make it upstream eventually?

What is preventing including this in the main nginx?

Seems rather useful!

@yaoweibin
Copy link
Owner

You can see the reason here: http://forum.nginx.org/read.php?2,225811,225822#msg-225822

@rngadam
Copy link
Author

rngadam commented Jan 15, 2013

Reading that thread, it sounds like FUD, no? Using rsyslogd, whether it is up or not, you can write to /dev/log. The only way to block the write is to suspend the rsyslogd thread... Plus, why couldn't this be made into a ./configure option and let users choose whether they want it or not?

Anyway, I'll be testing nginx + syslog patch + rsyslogd. If the results are positive I'll ping the mailing list and ask them to reconsider.

@yaoweibin
Copy link
Owner

Thanks. If you want to have a try, we have supported syslog in our nginx fork: http://tengine.taobao.org/index.html . We have implemented syslog protocol internally which don't block the socket at all.

@josephholsten
Copy link

syslog is included in nginx plus (per http://nginx.com/products/)

I'll be annoying on the official tracking bug again: http://trac.nginx.org/nginx/ticket/95

@josephholsten
Copy link

as soon as it lets me log in again.

@yaoweibin
Copy link
Owner

Could you try tengine(http://tengine.taobao.org/document/http_log.html)? It supports syslog natively. I'm now one of developer of tengine.

@josephholsten
Copy link

If be happy to! We're mostly using nginx for passenger. Does it support passenger out of the box?

http://josephholsten.com

On Dec 16, 2013, at 17:39, Weibin Yao(Ҧΰ±ó) notifications@github.com wrote:

Could you try tengine(http://tengine.taobao.org/document/http_log.html)? It supports syslog natively. I'm now one of developer of tengine.

¡ª
Reply to this email directly or view it on GitHub.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants