How to close the default geddy log system? #180

Closed
tony0918 opened this Issue Aug 30, 2012 · 7 comments

Comments

Projects
None yet
5 participants

If I doesn't need the default geddy log system,such as access log and error log,What can I do in the config file?Thanks.

Contributor

larzconwell commented Aug 31, 2012

Currently there's no way to disable Geddy's logging but I'm currently looking into implement it so that when make the logDir options null it won't write the log files.

Contributor

larzconwell commented Sep 2, 2012

Okay this is implemented in 6aec4d0 , if the logDir option in any of you environment config files is null or undefined then it will not write any files.

Thanks for the feature request!

tony0918 commented Sep 3, 2012

It's my pleasure.

For what it's worth, this doesn't work for me.
I get error on this line: var currentLog = path.join(dir, type + '.log')
claiming that only strings can be arguments to join
(which does make sense, i believe)

I'm also getting the error. I have the latest version (13.0.6 at this time of writing) installed.
path.js:204 throw new TypeError('Arguments to path.join must be strings');

Contributor

mde commented Oct 21, 2014

Sorry for this regression. Fixed in 5f2dc7b and pushed to NPM in v13.0.7.

Thank you for the fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment