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

SyslogHandling ErrorHandling #21

Open
emulgeator opened this issue May 24, 2015 · 3 comments
Open

SyslogHandling ErrorHandling #21

emulgeator opened this issue May 24, 2015 · 3 comments
Milestone

Comments

@emulgeator
Copy link
Collaborator

When a project is using Syslog for logging, and the syslog is down on the server for any reason it will result a 500 error in most of the cases.
Just stop syslog-ng on a dev server and test a project
The class should ignore these kind of errors in a way so it should not bring down the whole application

@emulgeator emulgeator modified the milestone: v0.4.0 May 25, 2015
@szeber
Copy link
Owner

szeber commented Jun 10, 2015

I'm not sure I agree with this one. It's the same in my opinion as if the database server went down.

We could potentially postpone this being an issue by deferring connection to the syslog socket till it's actually needed instead of it happening when the object is constructed, however syslog not being available, when you try to log something should still be an exception.

@emulgeator
Copy link
Collaborator Author

I know what you mean and I agree, of course it is crucial but not as essential as a db connection in most of the cases. I agree with you proposition to defer the connection , and I would also consider to make this extra behavior somehow configurable so I could tell that not being able to connect to syslog should kill the application or not.

@szeber
Copy link
Owner

szeber commented Jun 24, 2015

Yes, that configuration option sounds quite useful for example for a dev environment.

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

2 participants