fix: migrate Slack integration from Bot Token to User Token for security#185
Merged
breaking-brake merged 1 commit intomainbreaking-brake/cc-wf-studio:mainfrom Dec 1, 2025
feat/user-token-onlybreaking-brake/cc-wf-studio:feat/user-token-onlyCopy head branch name to clipboard
Merged
fix: migrate Slack integration from Bot Token to User Token for security#185breaking-brake merged 1 commit intomainbreaking-brake/cc-wf-studio:mainfrom feat/user-token-onlybreaking-brake/cc-wf-studio:feat/user-token-onlyCopy head branch name to clipboard
breaking-brake merged 1 commit intomainbreaking-brake/cc-wf-studio:mainfrom
feat/user-token-onlybreaking-brake/cc-wf-studio:feat/user-token-onlyCopy head branch name to clipboard
Conversation
- Remove Bot Token dependency, use User Token for all Slack API operations - Update OAuth flow to validate User Token instead of Bot Token - Remove Bot membership check (no longer needed with User Token) - Update token manager to check User Token for connection validation - Remove Bot Token input field from manual token dialog 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Merged
github-actions Bot
added a commit
that referenced
this pull request
Dec 1, 2025
## [2.17.2](v2.17.1...v2.17.2) (2025-12-01) ### Bug Fixes * migrate Slack integration from Bot Token to User Token ([#185](#185)) ([a6d2ce9](a6d2ce9))
Contributor
|
🎉 This PR is included in version 2.17.2 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
This was referenced Dec 1, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem
Security Issue
Users could post workflows to Slack channels they had left, as long as the Bot remained in those channels.
Current Behavior
Expected Behavior
Solution
Migrate all Slack API operations from Bot Token (
xoxb-) to User Token (xoxp-).Key Changes
Bot Token vs User Token:
chat:write): Posts as Bot, can post to any channel Bot is inchat:write): Posts as user, can only post to channels user is a member ofChanges
Extension Host:
slack-api-service.ts: RemoveensureClient()(Bot Token), use onlyensureUserClient()(User Token) for all operationsslack-connect-oauth.ts: Validate User Token instead of Bot Token in OAuth responseslack-connect-manual.ts: Only require User Token inputslack-token-manager.ts: Check User Token for connection validationopen-editor.ts: RemoveCHECK_BOT_CHANNEL_MEMBERSHIPhandlerWebview:
SlackShareDialog.tsx: Remove Bot membership check UISlackManualTokenDialog.tsx: Remove Bot Token input fieldslack-integration-service.ts: RemovecheckBotChannelMembership()functionTypes & i18n:
messages.ts: Remove Bot membership check typesImpact
Testing
🤖 Generated with Claude Code