search

movableink / doorman

HTTP Proxy + OAuth
MIT License
167 stars 52 forks source link

add options for logging #22

Closed kcrayon closed 9 years ago

kcrayon commented 9 years ago

I wanted to disable file logging in my setup, so I added these options.

This patch assumes #20 is merged, but that can easily be changed.

mnutt commented 9 years ago

Since I started doorman I've come to think that maybe winston isn't really necessary. If we were to just drop winston and always log to stdout, would that meet your use case?

kcrayon commented 9 years ago

That would work for me. Upstart, Docker and systemd all seem to prefer stdout logging.

mnutt commented 9 years ago

The latest version now just logs to stdout/stderr. stderr has the important info (startup configuration, actual errors) while stdout has request-level logging.