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

[proposal] Add an endpoint that crashes #70

Open
leplatrem opened this issue Aug 24, 2023 · 2 comments
Open

[proposal] Add an endpoint that crashes #70

leplatrem opened this issue Aug 24, 2023 · 2 comments

Comments

@leplatrem
Copy link
Contributor

leplatrem commented Aug 24, 2023

Adding a crashing endpoint to our services could help us make that the service monitoring (logging and graph of 5XX responses) and error reporting (Sentry, slack notifs) is configured correctly.

@app.route("/__crash__")
def crash():
    logger.error("about to boom")
    raise ValueError("boom")
@leplatrem
Copy link
Contributor Author

I see here that @jwhitlock and @willkg proposed a __broken__ endpoint for this

@willkg
Copy link

willkg commented Sep 5, 2023

I threw that confluence page together based on what @jwhitlock and I have done in the past because nothing existed and I needed something to point to. I'm game for changing the endpoint to a name that makes more sense.

Also, it should be behind basicauth or require a superuser so it doesn't allow people to spam our Sentry instance.

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

No branches or pull requests

2 participants