home-assistant / core

:house_with_garden: Open source home automation that puts local control and privacy first.
https://www.home-assistant.io
Apache License 2.0
69.03k stars 28.28k forks source link

Slack Web API method "files.upload" deprecated #116833

Closed philipp-frank closed 1 week ago

philipp-frank commented 1 week ago

The problem

The Slack integration uses the Slack Web API method files.upload.

Slack recently announced (https://api.slack.com/methods/files.upload):

files.upload is deprecated and will stop functioning on March 11, 2025. Use files.getUploadURLExternal and files.completeUploadExternal to upload files instead. Newly created apps will be unable to use files.upload beginning May 8, 2024.

What version of Home Assistant Core has the issue?

2024.5

What was the last working version of Home Assistant Core?

No response

What type of installation are you running?

Home Assistant OS

Integration causing the issue

Slack

Link to integration documentation on our website

https://www.home-assistant.io/integrations/slack/

Diagnostics information

No response

Example YAML snippet

No response

Anything in the logs that might be useful for us?

No response

Additional information

Source Code

home-assistant[bot] commented 1 week ago

Hey there @tkdrob, @fletcherau, mind taking a look at this issue as it has been labeled with an integration (slack) you are listed as a code owner for? Thanks!

Code owner commands Code owners of `slack` can trigger bot actions by commenting: - `@home-assistant close` Closes the issue. - `@home-assistant rename Awesome new title` Renames the issue. - `@home-assistant reopen` Reopen the issue. - `@home-assistant unassign slack` Removes the current integration label and assignees on the issue, add the integration domain after the command. - `@home-assistant add-label needs-more-information` Add a label (needs-more-information, problem in dependency, problem in custom component) to the issue. - `@home-assistant remove-label needs-more-information` Remove a label (needs-more-information, problem in dependency, problem in custom component) on the issue.

(message by CodeOwnersMention)


slack documentation slack source (message by IssueLinks)

ismarslomic commented 1 week ago

I can confirm that I did also receive e-mail notification from Slack about the deprecation of the files.upload web API, which is called by the Home Assistant Slack integration when sending file from local path to Slack (data.file.path).

We want to inform you of two important dates related to upcoming changes to support for the files.upload web API method that affect your Home Assistant app.

May 8, 2024 Any new applications created on or after this date will be prevented from invoking the files.upload API.

March 11, 2025 Support for the files.upload API will be discontinued.

To prepare for this change, we recommend migrating away from files.upload and instead using the combination of > files.getUploadURLExternal and > files.completeUploadExternal. More information on how > to use these two APIs together can be found on our [Uploading files documentation]> (https://api.slack.com/messaging/files#uploading_files). You can also leverage Slack’s SDKs to help you transition to > this new way of uploading files by visiting our api.slack page.

philipp-frank commented 1 week ago

Fix in https://github.com/home-assistant/core/pull/116868