-
-
Notifications
You must be signed in to change notification settings - Fork 299
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 status 500 instead of status 404 #110
Comments
@maremmle I submitted a fix in #112 does this solve your problem? |
@pxlpnk Yes it does log 404 for However for
|
This looks bad. I think I will make the handling a bit more explicit to only cat the errors we are aware of.
Any other exceptions that come to your mind? |
I can not reproduce your error @maremmle.
|
In case of an an
ActiveRecord::RecordNotFound
a status code500
gets logged while actually a404
response gets returned.If noticed that this is an issue for quite some time - according to #27. However, this thread is 2 years old now and it also addresses a more complex issue.
The text was updated successfully, but these errors were encountered: