Add extra infor for when scope is incoming-webhook

Slack just added a new scope that allows easy creation of incoming webhooks on authorization, this changes to the strategy allows it to return the new info slack provides to the client about the created webhook

Author: Gustavo Bazan

.gitignore

@@ -16,3 +16,5 @@ test/tmp
 test/version_tmp
 tmp
 .idea/
+.ruby-version
+.ruby-gemset

lib/omniauth-slack.rb

@@ -1,2 +1,2 @@
 require 'omniauth-slack/version'
-require 'omniauth/strategies/slack'
+require 'omniauth/strategies/slack'

lib/omniauth/strategies/slack.rb

@@ -3,14 +3,13 @@
 module OmniAuth
   module Strategies
     class Slack < OmniAuth::Strategies::OAuth2
+      option :name, 'slack'
 
-      option :name, "slack"
-
-      option :authorize_options, [ :scope, :team ]
+      option :authorize_options, [:scope, :team]
 
       option :client_options, {
-        site: "https://slack.com",
-        token_url: "/api/oauth.access"
+        site: 'https://slack.com',
+        token_url: '/api/oauth.access'
       }
 
       option :auth_token_params, {
@@ -43,19 +42,36 @@ class Slack < OmniAuth::Strategies::OAuth2
       end
 
       extra do
-        {:raw_info => raw_info, :user_info => user_info, :team_info => team_info}
+        {
+          raw_info: raw_info,
+          user_info: user_info,
+          team_info: team_info,
+          web_hook_info: web_hook_info
+        }
+      end
+
+      def raw_info
+        @raw_info ||= access_token.get('/api/auth.test').parsed
       end
 
       def user_info
         @user_info ||= access_token.get("/api/users.info?user=#{raw_info['user_id']}").parsed
       end
 
       def team_info
-        @team_info ||= access_token.get("/api/team.info").parsed
+        @team_info ||= access_token.get('/api/team.info').parsed
       end
 
-      def raw_info
-        @raw_info ||= access_token.get("/api/auth.test").parsed
+      def web_hook_info
+        return {} unless incoming_webhook_allowed?
+        access_token.params['incoming_webhook']
+      end
+
+      def incoming_webhook_allowed?
+        return false unless options['scope']
+        webhooks_scopes = ['incoming-webhook']
+        scopes = options['scope'].split(',')
+        (scopes & webhooks_scopes).any?
       end
     end
   end