search

kmrshntr / omniauth-slack

omniauth-slack
MIT License
95 stars 156 forks source link

Fix user_info method for Add to Slack #40

Open danielschwartz opened 8 years ago

danielschwartz commented 8 years ago

When going through an initial Add to Slack flow, you can't mix the identity.basic scope with other team-wide scopes being requested (like team:read or users:read). However, Slack does give us the currently signed in user_id inside our access_token.params object. So, if for whatever reason our user_identity['id'] is nil, we should grab the user_id from access_token.params so that we get the current users information instead of an error.

kevinelliott commented 7 years ago

Any reason this never got merged in?