Repository navigation
feat(standardized-auth): add an acme sample for standardized app auth - #305
Draft
WilliamBergamin wants to merge 3 commits into
Draft
WilliamBergamin wants to merge 3 commits into
WilliamBergamin wants to merge 3 commits into
Conversation
Connects user accounts with the Slack-owned Connect and Manage connection controls. Bolt's installation store holds each connection: the OAuth success hook reports it with apps.user.connection.requestAuth, the disconnect action forgets and revokes the user token, tokens_revoked forgets users who remove the app, and the user_connection_status_requested event is answered with apps.user.connection.update. App Home renders the auth_provider block. The user_connection request listener is written and tested but not yet registered, since no Bolt routes that payload to action listeners yet.
Resolves the conflict with #306, which replaced test.yml with ci.yml. Adds standardized-auth to the lint, typecheck, and test matrices. Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>
OAuth state no longer needs files on disk. The state is now an HMAC-signed token checked with the signing secret, and it expires after 5 minutes because a stateless token can be replayed until then. Installations stay in the file store so connections survive a restart. Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>
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
1 participant
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.
Summary
This pull request adds a
standardized-authsample: Acme, an app that connects user accounts with the Slack-owned Connect and Manage connection controls. Rocket skates, anvils, and giant magnets not included.apps.user.connection.requestAuth, which also redraws an open auth modal, and refreshes App Home.tokens_revokedforgets users who remove the app.user_connection_status_requestedis answered withapps.user.connection.update.auth_providerblock.handle_user_connection, for the Connect on the app profile, is written and tested but not registered yet, since Bolt does not route theuser_connectionpayload to.actionlisteners yet.slack_sdkto python-slack-sdk main by commit until the next release.Testing
In a workspace where standardized app authentication is available:
ngrok http 3000and replace1234-56-78-90-0.ngrok-free.appinmanifest.json.slack install --environment local, thenslack env initwith the client ID, client secret, signing secret, and your ngrok URL asBASE_URL.slack runand openhttps://<ngrok host>/slack/installto install the app.@you.🤖 Generated with Claude Code