Skip to content
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

Change debugging logging to use debug channel #949

Open
wants to merge 1 commit into
base: master
from

Conversation

@alexblackie
Copy link

@alexblackie alexblackie commented Dec 12, 2018

These messages are very specific to omniauth internals, and thus are
only useful when debugging or developing. Having these messages on
:info causes them to fill test suite output or other non-production
logs which are at the :info level. We should move these messages to
:debug so developers can still see them, but they do not pollute
non-development logs.

@vfonic
Copy link

@vfonic vfonic commented May 28, 2019

Indeed, I just came to this repo to figure out how to disable these logs. :)

Any ideas?

These messages are very specific to omniauth internals, and thus are
only useful when debugging or developing. Having these messages on
`:info` causes them to fill test suite output or other non-production
logs which are at the `:info` level. We should move these messages to
`:debug` so developers can still see them, but they to not pollute
non-development logs.
@alexblackie alexblackie force-pushed the alexblackie:noisy_logs branch from 0c94028 to f3ef5f8 Mar 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.