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

Catch Message.DoesNotExist error in message delete handler #39

Open
brizandrew opened this issue Jul 8, 2019 · 0 comments
Open

Catch Message.DoesNotExist error in message delete handler #39

brizandrew opened this issue Jul 8, 2019 · 0 comments

Comments

@brizandrew
Copy link
Contributor

If someone deletes a message in the database, and then deletes the message in Slack, Slack will send a message_delete event which will be handled. Unfortunately, if the message does not exist the handler error, responding to slack with a 500 error, which will make Slack retry the event repeatedly.

We need to catch the error in the handler, so that if the message does not exist, a 200 is returned anyway.

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

1 participant