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

Incorrect doc regarding logging #143

Closed
derks opened this Issue Jul 27, 2012 · 0 comments

Comments

Projects
None yet
1 participant
@derks
Member

derks commented Jul 27, 2012

In the following:

http://cement.readthedocs.org/en/latest/dev/logging/#logging-to-a-file

The last paragraph is incorrect. The log handler is setup one during app.setup(), which uses the name of the app as the logging namespace. Therefore all calls to app.log.info() for example will use the app name namespace... and not the current module where the log was called. Would be handy to change this to accept a name argument to override the app name... such as:

app.log.info('this is my log message', module=__name__)

This would maintain compatibility. Not sure if this should be built into the interface... or just supported by the LoggingLogHandler implementation.

@ghost ghost assigned derks Jul 27, 2012

derks pushed a commit that referenced this issue Jul 30, 2012

@derks derks closed this Jul 30, 2012

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