kmrshntr / omniauth-slack

omniauth-slack
MIT License
95 stars 154 forks source link

Add identity information to the "Add to Slack" flow. #56

Open kevinelliott opened 7 years ago

kevinelliott commented 7 years ago

It is well known that Slack requires identity scopes to be separate from additional scopes. This typically requires pushing a user down the OAuth flow twice (once for "Sign in with Slack" and one with "Add to Slack").

This solution addresses that by gathering the information normally in the identity scopes from the "Add to Slack" flow by calling the auth.test and users.info APIs and stuffing the data conditionally.

Also, allow redirect_uri to be defined on the configuration.

I would like to see this get merged into master.