Skip to content

Cherry-pick 2a409bbba: fix(slack): replace files.uploadV2 with 3-step upload#1430

Merged
alexey-pelykh merged 1 commit intomainfrom
cherry-pick/2a409bbba-slack-upload-flow
Mar 15, 2026
Merged

Cherry-pick 2a409bbba: fix(slack): replace files.uploadV2 with 3-step upload#1430
alexey-pelykh merged 1 commit intomainfrom
cherry-pick/2a409bbba-slack-upload-flow

Conversation

@alexey-pelykh
Copy link
Copy Markdown

Cherry-pick of upstream commit 2a409bbba — "fix(slack): replace files.uploadV2 with 3-step upload flow to fix missing_scope error (openclaw#17558)"

Conflicts resolved:

  • CHANGELOG.md — removed (deleted in fork)

Part of #677

@alexey-pelykh alexey-pelykh force-pushed the cherry-pick/2a409bbba-slack-upload-flow branch from de5949b to 1891dd2 Compare March 15, 2026 14:37
…sing_scope error (openclaw#17558)

* fix(slack): replace files.uploadV2 with 3-step upload flow

files.uploadV2 from @slack/web-api internally calls the deprecated
files.upload endpoint, which fails with missing_scope even when
files:write is correctly granted in the bot token scopes.

Replace with Slack's recommended 3-step upload flow:
1. files.getUploadURLExternal - get presigned URL + file_id
2. fetch(upload_url) - upload file content
3. files.completeUploadExternal - finalize & share to channel/thread

This preserves all existing behavior including thread replies via
thread_ts and caption via initial_comment.

* fix(slack): harden external upload flow and tests

---------

Co-authored-by: Tak Hoffman <[email protected]>
(cherry picked from commit 2a409bb)
@alexey-pelykh alexey-pelykh force-pushed the cherry-pick/2a409bbba-slack-upload-flow branch from 1891dd2 to d274a7a Compare March 15, 2026 14:47
@alexey-pelykh alexey-pelykh merged commit 02418df into main Mar 15, 2026
7 checks passed
@alexey-pelykh alexey-pelykh deleted the cherry-pick/2a409bbba-slack-upload-flow branch March 15, 2026 14:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants