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.

Sign up for GitHub

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

Jump to bottom

Sentry on NAU Richie is raising error of any missing page http status 404 #220

Closed
igobranco opened this issue Apr 23, 2024 · 4 comments
Closed

Sentry on NAU Richie is raising error of any missing page http status 404 #220

igobranco opened this issue Apr 23, 2024 · 4 comments
Assignees
@igobranco

Comments

@igobranco
Copy link
Member

igobranco commented Apr 23, 2024 •
edited
Loading

Any missing page on the NAU Richie site is being logged as a log error, so sentry is sending it has an error.

So, for example, a broken link not correct shared on Facebook is being crawled by Facebook bot, and is being reported by Sentry as an error.

In general a 404 is a warning and not an error.

We need to configure Django, on NAU Richie, to log a 404 as a warning and not as an error.

Image
@igobranco igobranco self-assigned this Apr 23, 2024
@igobranco
Copy link
Member Author

Possible solution could be disable the request.summary logger on Django with level error.

@igobranco
Copy link
Member Author

This logging is being written by dockerflow package on https://github.com/mozilla-services/python-dockerflow/blob/0424734c268aea36312117335b92ea79bff2c8ee/src/dockerflow/django/middleware.py#L85-L90

@igobranco igobranco mentioned this issue Apr 24, 2024
Open
igobranco added a commit that referenced this issue Apr 24, 2024
@igobranco
🔧(sentry) ignore dockerflow request.summary logger
7b30f9b 
Configure Sentry to ignore the dockerflow `request.summary` logger.
This logger logs as ERROR any 404.
With so many crawlers getting incorrect paths, this is
too much errors on Sentry.
relates to #220
igobranco added a commit that referenced this issue Apr 24, 2024
@igobranco
🔊(log) remove request.summary logger
d655e81 
The dockerflow `request.summary` logger logs
any 404 with ERROR level. This is too verbose.
It logs a big JSON.
relates to #220
igobranco added a commit that referenced this issue Apr 24, 2024
@igobranco
🔊(log) remove request.summary logger
a01f653 
The dockerflow `request.summary` logger logs
any 404 with ERROR level. This is too verbose.
It logs a big JSON.
relates to #220
igobranco added a commit that referenced this issue Apr 24, 2024
@igobranco
🔊(log) remove request.summary logger
7363350 
The dockerflow `request.summary` logger logs
any 404 with ERROR level. This is too verbose.
It logs a big JSON.
relates to #220
igobranco added a commit that referenced this issue Apr 24, 2024
@igobranco
🔧(sentry) ignore dockerflow request.summary logger
da20106 
Configure Sentry to ignore the dockerflow `request.summary` logger.
This logger logs as ERROR any 404.
With so many crawlers getting incorrect paths, this is
too much errors on Sentry.
relates to #220
igobranco added a commit that referenced this issue Apr 24, 2024
@igobranco
🔊(log) remove request.summary logger
56b6246 
The dockerflow `request.summary` logger logs
any 404 with ERROR level. This is too verbose.
It logs a big JSON.
relates to #220
@igobranco
Copy link
Member Author

Pending deploy

@igobranco
Copy link
Member Author

Generated a new NAU release, deployed in PROD.

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

@igobranco igobranco

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@igobranco