From 0e57d7d497e24067f03805c9bc906bfd66d0898d Mon Sep 17 00:00:00 2001
From: Weves <chrisweaver101@gmail.com>
Date: Thu, 13 Mar 2025 18:28:57 -0700
Subject: [PATCH] Loud error on slack doc sync missing permissions
---
backend/ee/onyx/external_permissions/slack/doc_sync.py | 10 +++++++---
1 file changed, 7 insertions(+), 3 deletions(-)
diff --git a/backend/ee/onyx/external_permissions/slack/doc_sync.py b/backend/ee/onyx/external_permissions/slack/doc_sync.py
index 86f3ec9f9..ce8b883a2 100644
--- a/backend/ee/onyx/external_permissions/slack/doc_sync.py
+++ b/backend/ee/onyx/external_permissions/slack/doc_sync.py
@@ -117,9 +117,7 @@ def _get_slack_document_access(
if callback:
if callback.should_stop():
- raise RuntimeError(
- "_get_slack_document_access: Stop signal detected"
- )
+ raise RuntimeError("_get_slack_document_access: Stop signal detected")
callback.progress("_get_slack_document_access", 1)
@@ -138,6 +136,12 @@ def slack_doc_sync(
token=cc_pair.credential.credential_json["slack_bot_token"]
)
user_id_to_email_map = fetch_user_id_to_email_map(slack_client)
+ if not user_id_to_email_map:
+ raise ValueError(
+ "No user id to email map found. Please check to make sure that "
+ "your Slack bot token has the `users:read.email` scope"
+ )
+
workspace_permissions = _fetch_workspace_permissions(
user_id_to_email_map=user_id_to_email_map,
)