-
-
Notifications
You must be signed in to change notification settings - Fork 5.3k
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
don't record error when request a non-exist user #15819
Conversation
Co-authored-by: zeripath <art27@cantab.net>
If there a particular common user that is being reported because of this e.g. serviceworker.js then applying this will mean that we're hiding an actual performance problem. For example - we shouldn't be trying to get serviceworker.js from the db in the first place as it's an invalid username. |
I think some router tests should resolve the problem. This PR is to prevent to log code stack when a non-exist user. |
🚀 |
* don't record error when request a non-exist user * Update routers/repo/http.go Co-authored-by: zeripath <art27@cantab.net> Co-authored-by: zeripath <art27@cantab.net> Co-authored-by: techknowlogick <techknowlogick@gitea.io>
As title.