Skip to content

Commit dd2dd9a

Browse files
committed
bridge_with_slack: Do not use a stale Zulip client for send_message.
1 parent 7bebd7d commit dd2dd9a

File tree

2 files changed

+11
-6
lines changed

2 files changed

+11
-6
lines changed

zulip/integrations/bridge_with_slack/run-slack-bridge

+10-6
Original file line numberDiff line numberDiff line change
@@ -53,11 +53,15 @@ class SlackBridge:
5353
}
5454

5555
# zulip-specific
56-
self.zulip_client = zulip.Client(
57-
email=self.zulip_config["email"],
58-
api_key=self.zulip_config["api_key"],
59-
site=self.zulip_config["site"],
60-
)
56+
def zulip_client_constructor() -> zulip.Client:
57+
return zulip.Client(
58+
email=self.zulip_config["email"],
59+
api_key=self.zulip_config["api_key"],
60+
site=self.zulip_config["site"],
61+
)
62+
63+
self.zulip_client = zulip_client_constructor()
64+
self.zulip_client_constructor = zulip_client_constructor
6165

6266
# slack-specific
6367
self.slack_client = rtm
@@ -121,7 +125,7 @@ class SlackBridge:
121125
subject=zulip_endpoint["topic"],
122126
content=content,
123127
)
124-
self.zulip_client.send_message(msg_data)
128+
self.zulip_client_constructor().send_message(msg_data)
125129

126130
self.slack_client.start()
127131

zulip_bots/zulip_bots/bots/stack_overflow/stack_overflow.py

+1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
import logging
22
from typing import Dict, Final, Optional
3+
||||||| Stash base
34

45
import requests
56

0 commit comments

Comments
 (0)