Skip to:

  1. Skip to Jira Navigation
  2. Skip to Side Navigation
  3. Skip to Main Content

Update documentation (or implementation) for newer Slack API

Description

As discussed on discourse, Slack’s new ‘Slack App’ webhook API doesn’t allow setting the channel. I posted a method that works for substituting the webhook url in the notification command in that thread. The documentation should be updated to note this limitation (and possibly to recommend a way to handle things) - instead of the previously recommended method from NMS-10628). Alternatively, the implementation could be updated to support slack’s bot user API w/ oauth and chat.postMessage (which can be used to specify a channel).

BR

For documentation

In the slack notifications chapter (https://docs.opennms.com/horizon/latest/operation/deep-dive/notifications/strategies/slack.html), provide the appropriate text regarding the fact that you can’t specify the channel to send in the new slack API, and include the method described in the discourse article listed above.

Acceptance / Success Criteria

None

Attachments

1
  • 06 Jul 2023, 07:15 AM

Activity

Show:

Marcel Fuhrmann July 6, 2023 at 7:15 AM

When it is about the old webhook integration: Slack tends to use rate limits. So it can happen, that you don’t get notifications anymore and instead of your notifications you get:

I’ve never seen this before in multiple years of usage. So I assume they try to push the migration to “Slack Apps” more to get rid of this old webhook feature.

Bonnie Robinson June 21, 2023 at 12:38 PM

Merged. Thanks for the solution and ticket, and helping to improve the documentation!

Bonnie Robinson June 19, 2023 at 9:34 PM

Bonnie Robinson June 6, 2023 at 12:47 PM

I think it’s a good idea to have it in the documentation. For fixing the existing Slack integration, would suggest creating a ticket and let the Horizon team see it in the backlog to determine if they can fix it.

Marcel Fuhrmann June 6, 2023 at 12:21 PM

What do you think about a change in the docs here? This workaround is really important and luckily we have it documented in Discourse already. But it feels to me, this issue should be about fixing the existing Slack integration in the code.

Fixed

Details

Assignee

Reporter

Labels

HB Grooming Date

HB Backlog Status

Original estimate

Time tracking

No time logged2h remaining

Components

Sprint

Fix versions

Affects versions

Priority

PagerDuty

Created May 5, 2023 at 5:21 PM
Updated May 21, 2024 at 10:10 AM
Resolved June 21, 2023 at 12:38 PM

Flag notifications