Merge pull request #827 from seratch/issue-629-remove-confusing-debug…

…-logs Fix #629 confusing debug log by ConversationStore
slackapi · Mar 10, 2021 · b8bc037 · b8bc037
2 parents 257dd6d + 4e9a676
commit b8bc037
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/src/conversation-store.ts b/src/conversation-store.ts
@@ -62,9 +62,12 @@ export function conversationContext<ConversationState = any>(
       context.updateConversation = (conversation: ConversationState) => store.set(conversationId, conversation);
       try {
         context.conversation = await store.get(conversationId);
-        logger.debug(`Conversation context loaded for ID ${conversationId}`);
+        logger.debug(`Conversation context loaded for ID: ${conversationId}`);
       } catch (error) {
-        logger.debug(`Conversation context failed loading for ID: ${conversationId}, error: ${error.message}`);
+        if (error.messsage !== 'undefined' && error.message !== 'Conversation not found') {
+          // The conversation data can be expired - error: Conversation expired
+          logger.debug(`Conversation context failed loading for ID: ${conversationId}, error: ${error.message}`);
+        }
       }
     } else {
       logger.debug('No conversation ID for incoming event');