api/cannon: Stop retrying invalid webhooks forever

[victorges]

What does this pull request do? Explain your changes. (required)

Turns out there were never any suspended users sending webhooks. Only webhooks
that never got acked from the queue after they got suspended.

When we got an invalid webhook for a deleted or suspended user or stream,
we threw an error from the fucntion that processes the events. Problem was the
caller of that function was retrying on any generic errors.

Made them throw an explicit UnprocessableEntityError instead so we drop
the messages accordingly.

Specific updates (required)

• Throw UnprocessableEntityError from main process func
• Handle that error and make sure we ack messages from them
• Remove explicitly handling for recording.waiting events, we can just rethrow now

How did you test each of these updates (required)
yarn test

Does this pull request close any open issues?
Not sure, we talked about it tho and the webhook queue is never empty.

Checklist

• I have read the CONTRIBUTING document.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have added tests to cover my changes.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
livepeer-studio	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	May 6, 2024 0:53am

[gioelecerati]

LGTM