Fixed 500 error message #2648

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
7 participants

god commented Aug 23, 2011

Fixed 500 error message which is misleading by suggesting all rails apps have exception notifications by default. This discourages users from complaining when the site fails for them.

Member

vijaydev commented Aug 23, 2011

My 2 cents: The proposed message will make users wonder how to notify site owners! The existing message expects users to trust that the site owners are responsible (which all developers should be, imho).

god commented Aug 23, 2011

Fair enough, I would just propose getting rid of that line entirely. Don't explicitly discourage the user from contacting the site owner by showing that the site owner was notified (which isn't always true). Just show that there was an error.

Contributor

tkrajcar commented Aug 24, 2011

Agreed with this pull request and god's comment. Just delete the entire line.

Contributor

lsylvester commented Aug 24, 2011

+1 for deleting the line.

Contributor

siong1987 commented Aug 27, 2011

Just create a issue #2713 that removed the entire line.

josevalim added a commit that referenced this pull request Sep 5, 2011

Merge pull request #2713 from FLOChip/500-fixed
remove misleading line on 500 error page. #2648

@josevalim josevalim closed this Sep 5, 2011

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