Make logging optional and replaceable #10

Closed
gcao opened this Issue Oct 9, 2012 · 2 comments

Comments

Projects
None yet
2 participants
Owner

gcao commented Oct 9, 2012

Provide a default logger but make it possible to be replaced.

logger interface

initialize context, options = {}

level

level=

visible? level

log level, *args

log levels:
error - 50
warn - 40
info - 30
debug - 20
trace - 10

Collaborator

mensfeld commented Jul 6, 2015

can't we use a std ruby logger and its std interface? it is common to use this approach with other loggers so aspector would integrate automatically with any logger there is (like rails logger ,etc)

Owner

gcao commented Jul 6, 2015

I've not looked at the standard logger before implementing this feature. What I would like to do is to be able to change log level externally. If the standard logger supports that, we can switch to use it.

I'll close this as it's done. Feel free to open a new issue to switch logger implementation

gcao closed this Jul 6, 2015

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