ueberauth / ueberauth_slack

Slack OAuth2 Strategy for Überauth
MIT License
23 stars 33 forks source link

if default_scopes doesn't have 'users.read', authentication callback doesn't work #17

Open prashanthsadasivan opened 7 years ago

prashanthsadasivan commented 7 years ago

Took forever to track this one down. I don't really need 'users:read' for my application, but my authentication wasn't working because the strategy tries to make the call to users.info even though it doesn't exist in the scope. pull request incoming

cjab commented 6 years ago

I ran into this also ☹️. Just adding that users:read is required to the documentation would probably help others to avoid the problem 😃. Or it could always be appended to the scope list regardless of the defaults set in config.

doomspork commented 6 years ago

@cjab would you like to open a PR to update the README? There's an outstanding PR but it's gone stale.