[go: up one dir, main page]

Skip to content

Handle failed requests when posting notifications to Slack/Mattermost

Everyone can contribute. Help move this issue forward while earning points, leveling up and collecting rewards.

Summary

With !86168 (merged) we started logging error responses from the Slack/Mattermost API when posting notifications in app/models/concerns/integrations/slack_mattermost_notifier.rb.

Some of them are rate limit responses (HTTP 429 errors) which we plan to handle in #356896, but we should investigate other errors too and decide how to handle them.

Improvements

Errors can be seen at https://log.gprd.gitlab.net/goto/76be1c00-cb77-11ec-b73f-692cc1ae8214

Some of them like team_disabled and channel_not_found could be handled by disabling the integration, and maybe notifying users somehow.

Edited by 🤖 GitLab Bot 🤖