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

logging: Skip log records originated in site packages. #4076

Closed
wants to merge 1 commit into from

Conversation

umairwaheed
Copy link
Member

This fixes the huge exception we get in our logs from django.template
logger. This exception is a known bug in Django, see
https://code.djangoproject.com/ticket/26886

Fixes #3974

@timabbott, please review.

This fixes the huge exception we get in our logs from django.template
logger. This exception is a known bug in Django, see
https://code.djangoproject.com/ticket/26886

Fixes zulip#3974
@zulipbot
Copy link
Member

Hello @zulip/server-tooling members, this pull request references an issue with the "area: tooling" label, so you may want to check it out!

@smarx
Copy link

smarx commented Mar 15, 2017

Automated message from Dropbox CLA bot

@umairwaheed, it looks like you've already signed the Dropbox CLA. Thanks!

@timabbott
Copy link
Sponsor Member

Merged, after tweaking the comment for the new function to mention it's a workaround for that Django bug.

@timabbott timabbott closed this Mar 15, 2017
@umairwaheed umairwaheed deleted the 3974-exception branch August 14, 2017 04:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

run-dev.py crazy traceback in URL resolvers
4 participants