Merge pull request #1 from practo/slack-fix

Slack fix
practo · Feb 20, 2019 · 2418019 · 2418019
2 parents 28d497f + 642e752
commit 2418019
Show file tree

Hide file tree

Showing 4 changed files with 9 additions and 4 deletions.
diff --git a/CHANGES b/CHANGES
@@ -1,3 +1,8 @@
+Version 9.0.0-practo1
+-----------
+
+Fixes broken Slack integration due to the workspace apps preview feature reverted by Slack
+
 Version 9.0
 -----------
 

diff --git a/setup.py b/setup.py
@@ -48,7 +48,7 @@
 )
 
 # The version of sentry
-VERSION = '9.0.0'
+VERSION = '9.0.0-practo1'
 
 # Hack to prevent stupid "TypeError: 'NoneType' object is not callable" error
 # in multiprocessing/util.py _exit_function when running `python

diff --git a/src/sentry/identity/slack/provider.py b/src/sentry/identity/slack/provider.py
@@ -11,7 +11,7 @@ class SlackIdentityProvider(OAuth2Provider):
     # TODO(epurkhiser): This identity provider is actually used for authorizing
     # the Slack application through their Workspace Token OAuth flow, not a
     # standard user access token flow.
-    oauth_access_token_url = 'https://slack.com/api/oauth.token'
+    oauth_access_token_url = 'https://slack.com/api/oauth.access'
     oauth_authorize_url = 'https://slack.com/oauth/authorize'
 
     oauth_scopes = (

diff --git a/src/sentry/integrations/slack/integration.py b/src/sentry/integrations/slack/integration.py
@@ -44,7 +44,7 @@ class SlackIntegrationProvider(IntegrationProvider):
         'channels:read',
         'groups:read',
         'users:read',
-        'chat:write',
+        'chat:write:bot',
         'links:read',
         'links:write',
         'team:read',
@@ -100,7 +100,7 @@ def build_integration(self, state):
             },
             'user_identity': {
                 'type': 'slack',
-                'external_id': data['authorizing_user_id'],
+                'external_id': data['user_id'],
                 'scopes': [],
                 'data': {},
             },