Repository navigation
feat(standardized-auth): add an acme sample for standardized app auth - #95
Open
WilliamBergamin wants to merge 9 commits into
Open
WilliamBergamin wants to merge 9 commits into
WilliamBergamin wants to merge 9 commits into
Conversation
Connects user accounts with the Slack-owned Connect and Manage connection controls. Bolt's installation service 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. This depends on java-slack-sdk 1.52.1-SNAPSHOT (AuthProviderBlock, the new methods, and the status-requested event model) until it is released.
Match slack-samples/bolt-python-examples#305: run over Socket Mode, keep connections in a local properties file, handle the user_connection event and the fixed auth_provider_connect and auth_provider_disconnect action IDs, and answer user_connection_status_requested. The OAuth flow, installation service, tokens_revoked, and token revocation are gone. The SDK has no user_connection event model yet, so the sample defines UserConnectionEvent with a null default subtype to match Bolt's routing key. Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>
Assert Bolt keys UserConnectionEvent as user_connection, and restore the connections path before deleting the temp directory. Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>
… sample The Java client returns ok:false responses without throwing, and Bolt logs nothing at slf4j-simple's default INFO level, so a failed requestAuth left no trace. Raise com.slack.api to debug so the SDK's DetailedLoggingListener prints each call and its response, with the Authorization header redacted. Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>
…runs on Note that slack run starts the app with mvn compile exec:java from the start hook in .slack/hooks.json, and point to the debug logging config. Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>
java-slack-sdk#1660 now adds UserConnectionEvent, so drop the sample's stand-in and its test. MainTest keeps the routing check that one listener receives both the connect and disconnect subtypes. Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>
Swap slf4j-simple for logback-classic 1.6.5 and copy the starter template's logback.xml, which keeps com.slack.api at debug so every Slack API call and its response is logged. Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>
The next java-slack-sdk minor ships requestAuth and the user_connection models. CI fails to resolve until 1.53.0 is on Maven Central. 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
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 lets users connect their account with Slack's built-in Connect and Manage connection controls. Rocket skates, anvils, and giant magnets not included.Before merging
Needs Bolt for Java
1.53.0to be released. The sample's CI job fails until then.Testing
Use a workspace with standardized app authentication and Slack CLI v4.11.0 or newer.
slack runinstandardized-auth.🤖 Generated with Claude Code