awx
awx copied to clipboard
Improve Slack Notification Error Handling
ISSUE TYPE
- Enhancement
COMPONENT NAME
- API
- UI
SUMMARY
Currently we only surface errors when sending slack notification if a Python exception occurs.
We could do better if we read the event stream after our rtm_send_message
call to see if there
were any error events waiting.
A common error is when the bot has not been invited to a channel that we are attempting to send the notification to. This can be caught in the message stream, but does not throw an exception.
ADDITIONAL INFORMATION
https://github.com/ansible/awx/blob/devel/awx/main/notifications/slack_backend.py#L49
Hi @wwitzel3, I would like to take up this issue. Can you please assign it to me?
No one is actively working on it. Just do the work and submit a PR