Skip to content
This repository has been archived by the owner on Jul 14, 2023. It is now read-only.

show an error page on 500. #151

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

show an error page on 500. #151

wants to merge 1 commit into from

Conversation

fiatjaf
Copy link
Contributor

@fiatjaf fiatjaf commented Jun 14, 2017

For a long time we've had a "500.html" template that seated uselessly in the codebase. Not anymore!

Changes proposed in this pull request:

Have you made sure to add:

  • Tests
  • Documentation

Screenshots and GIFs

screenshot-error

@rohitdatta
Copy link
Member

I think that instead of just showing the stack trace, we should generate a unique ID, place on the page the current time and unique ID, with a request for them to email us these pieces of information.

We then put the unique ID and stack trace into the logs to avoid scaring the user (which could not even be the original site developer but a user) with a stack trace.

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