Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

slack error in superset #31196

Closed
3 tasks done
Chethanibms opened this issue Nov 29, 2024 · 5 comments
Closed
3 tasks done

slack error in superset #31196

Chethanibms opened this issue Nov 29, 2024 · 5 comments
Labels
alert-reports Namespace | Anything related to the Alert & Reports feature

Comments

@Chethanibms
Copy link

Bug description

slack error in superset

Screenshots/recordings

Screenshot 2024-11-29 at 1 45 57 PM

Superset version

master / latest-dev

Python version

3.9

Node version

16

Browser

Chrome

Additional context

superset.commands.report.exceptions.ReportScheduleClientErrorsException: [SupersetError(message="The request to the Slack API failed. (url: https://www.slack.com/api/files.upload)\nThe server responded with: {'ok': False, 'error': 'method_deprecated'}", error_type=<SupersetErrorType.REPORT_NOTIFICATION_ERROR: 'REPORT_NOTIFICATION_ERROR'>, level=<ErrorLevel.WARNING: 'warning'>, extra=None)]

Checklist

  • I have searched Superset docs and Slack and didn't find a solution to my problem.
  • I have searched the GitHub issue tracker and didn't find a similar bug report.
  • I have checked Superset's logs for errors and if I found a relevant Python stacktrace, I included it here as text in the "additional context" section.
Copy link

dosubot bot commented Nov 29, 2024

Hey there, @Chethanibms! 👋 I'm here to help you with any bugs, questions, or contributions you have for the project. Let's squash those bugs together!

A similar closed issue was found regarding a Slack error in Superset. The solution involves updating 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 in the Superset codebase. The relevant code can be found at superset/reports/notifications/slack.py. Additionally, ensure that the recipients in Superset for Slack are channel IDs and not usernames [1].

To continue talking to Dosu, mention @dosu.

@dosubot dosubot bot added the alert-reports Namespace | Anything related to the Alert & Reports feature label Nov 29, 2024
@geido
Copy link
Member

geido commented Dec 2, 2024

Hello @Chethanibms please check this PR and make sure you have the correct scopes in place #29264. Let me know if you will still be experiencing this problem. For now, I'll close this issue. Thanks!

@geido geido closed this as completed Dec 2, 2024
@czapajew
Copy link

I'm using the same issue after installing version 4.1.1 with helm chart.

@czapajew
Copy link

Can we reopen this one?

@rusackas
Copy link
Member

@czapajew can you give us a little more to go on? Does reviewing/setting the correct scopes as @geido suggested solve the issue?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
alert-reports Namespace | Anything related to the Alert & Reports feature
Projects
None yet
Development

No branches or pull requests

4 participants