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

Rails ActionController::RoutingError Not Being Reported As Default #820

Open
berniechiu opened this issue Mar 26, 2024 · 2 comments
Open
Labels
backlog We hope to fix this feature/bug in the future bug Confirmed bug

Comments

@berniechiu
Copy link

berniechiu commented Mar 26, 2024

Describe the bug

Rails ActionController::RoutingError not being reported, others look normal as expected.

Environment

  • Bugsnag version: 6.25.2
  • Ruby version: 3.2.3
  • Bundle version: 2.4.19
  • Integration framework version:
    • Rails: 7.0.8

Example code snippet

Bugsnag.configuration.ignore_classes
# => #<Set: {SystemExit, SignalException}>
Error messages:
# Server Logs
# ActionController::RoutingError (No route matches [GET] "/storefront/ba_tracking.js-1234567.map"
Screen Shot 2024-03-26 at 3 31 49 PM
@mclack
Copy link

mclack commented Apr 16, 2024

Hi @berniechiu

Thanks for your patience here.

In Rails, we capture errors as they are raised through the Rack server. If a request raises an error, we capture and notify it.
After looking into this, it appears ActionController::RoutingError is not actually being raised through Rack. It's instead responding with the error page and not raising an error, presumably to keep the application running as it hasn't entered a failure state (which 4xx errors traditionally aren't).

We have now added a task to our backlog to look into changing this, so that these errors are able to be automatically reported. I can't currently give an ETA on when this could be implemented, but if you explicitly raise a RoutingError in the meantime, it should be caught and notified.

We'll make sure to update this thread with any developments around automatic reporting for these errors.

@mclack mclack added bug Confirmed bug backlog We hope to fix this feature/bug in the future labels Apr 16, 2024
@berniechiu
Copy link
Author

Thanks a lot

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backlog We hope to fix this feature/bug in the future bug Confirmed bug
Projects
None yet
Development

No branches or pull requests

2 participants