Skip to content
This repository has been archived by the owner on Jan 23, 2024. It is now read-only.

Errship Gem Not working #45

Closed
ryanatwork opened this issue May 4, 2012 · 6 comments · Fixed by #116
Closed

Errship Gem Not working #45

ryanatwork opened this issue May 4, 2012 · 6 comments · Fixed by #116

Comments

@ryanatwork
Copy link
Contributor

Errship gem should display the correct error page when an error occurs but it's not happening

Example

If you load up http://brigade.codeforamerica.org/locations the 404 page should render but instead the default Rails 404 renders
If you go to http://brigade.codeforamerica.org/notfound then the correct 404 page shows

This also happens with 500 pages

http://brigade.codeforamerica.org/error
versus
http://brigade.codeforamerica.org/deployed_applications
Which renders the default Rails 500 page instead of the nice 500 page.

@mchung
Copy link
Contributor

mchung commented Dec 6, 2012

Due to a change in the way routing is handled in Rails 3.2, we may no longer depend on rescue_from AbstractController::ActionNotFOund to catch the exception and redirect to a 404.

The fix is to configure exceptions_app so that errors are also routed:

https://github.com/vidibus/vidibus-routing_error#alternative-one.

I can submit a patch later if this sounds good.

@kmcurry
Copy link
Contributor

kmcurry commented Dec 11, 2012

I'm not well qualified to decide. Any second opinions out there?

@rmcastil
Copy link
Contributor

@mchung sure. can you submit a pull request and we'll check it out?

@kmcurry
Copy link
Contributor

kmcurry commented Jan 3, 2013

I'm not able to verify pull request #116 on localhost (or even issue #45). It could be a problem with my local server. When I point to /locations and /notfound I get responses like:

Routing Error

No route matches [GET] "/notfound"
Try running rake routes for more information on available routes.

@kmcurry kmcurry closed this as completed in 8312352 Jan 4, 2013
@rmcastil
Copy link
Contributor

rmcastil commented Jan 7, 2013

@kmcurry Sorry I've been missing in action. Had a crazy holiday. Did you just close this pull request?

@kmcurry
Copy link
Contributor

kmcurry commented Jan 7, 2013

yep!

On Mon, Jan 7, 2013 at 9:22 AM, Ryan Castillo notifications@github.comwrote:

@kmcurry https://github.com/kmcurry Sorry I've been missing in action.
Had a crazy holiday. Did you just close this pull request?


Reply to this email directly or view it on GitHubhttps://github.com//issues/45#issuecomment-11953095.

Kevin Curry
Program Director
CfA Brigade
757.613.8158
GMT-5
http://brigade.codeforamerica.org

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants