Slack Notification - deprecated API about superset HOT 7 CLOSED

Hey there, @PuszekSE! I'm here to help you with any bugs, questions, or contributions you have. Let's work together to resolve this issue.

To resolve the issue of sending notifications via Slack failing due to a deprecated Slack API method in your Superset deployment, you should update your Slack app to use the new API methods. Specifically, you should replace the deprecated files_upload method with the new files_upload_v2 method.

Here is the relevant part of the code that shows the use of the new method:

# files_upload returns SlackResponse as we run it in sync mode.
for channel in channels:
    if len(files) > 0:
        for file in files:
            client.files_upload_v2(
                channel=channel,
                file=file,
                initial_comment=body,
                title=title,
            )
    else:
        client.chat_postMessage(channel=channel, text=body)

Ensure that your Slack app has the necessary permissions and scopes to use the new API methods. If the new method fails, the code falls back to the deprecated method, but this should be avoided by updating your Slack app configuration ^[1].

To continue talking to Dosu, mention @dosu.

from superset.

Where can i find this code to replace it in superset files?

@dosu

from superset.

I think this has recently been patched on master. @eschutho would know the most here, I think.

from superset.

I've checked this on 4.0.2-dev, 1-2 days after its release (although that isn't an option present on the version dropdown list).

from superset.

changing the function in superset/reports/notifications/slack.py. from files_upload to files_upload_v2 worked for me. Also the recipients in superset for slack should be channel IDs and not usernames.

from superset.

#29264
#29264

I believe those were related PRs

from superset.

Correct, the above two prs will fix this issue. There's one outstanding issue to be merged to fix the backend runtime migrations: #29721

from superset.