-
-
Notifications
You must be signed in to change notification settings - Fork 115
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
Refactor MinimalLogger / Disable by Default #612
Comments
Triggering Cement logging based on |
Hi @derks just to get in touch here with you, I do not understand your idea. Let's try to state something out: a. You want to make logging configurable My question is why you want to want to differ between framework and app logging Isn't it that you want debug level overall if you you need? For my opinion the framework should be suggested as being stable - So for an app developer it is not a point to LOG something from framework in anyway. But if the framework decide to print some debug logs it has to be noticed by the app developer as well. Saying that I think, that the number of LOG output from framewrk should be very rare. But if they are set, they should appear in any way. So if one enables DEBUG, it should printout also the framework logs. And if I configure a special logger, I want to have all logs catched by it. So for that, I would prefer to only log via app.log and only if set. If someone does not set a logger, than ther should be no log. My 2 cents on that |
The minimal logger is documented in multiple places as the "framework logger" and "should only be used by the framework". This is also one of the few places where a hardcoded look for
--debug
is. I want to disable framework logging by default, and remove the look for--debug
... as this should really only be used by core and extensions and should only log DEBUG... backward compatibility should not be an issue.App.meta.framework_logging = False
Related:
Notes:
CEMENT_FRAMEWORK_LOGGING
for backward-compat.The text was updated successfully, but these errors were encountered: