websocket_client-based SocketModeClient does not reconnect after a DNS outage

[ktindall-godaddy]

Reproducible in:

slack_sdk==3.26.2
Python 3.9.18
ProductName:            macOS
ProductVersion:         13.6.4
BuildVersion:           22G513
Darwin Kernel Version 22.6.0: Sun Dec 17 22:13:25 PST 2023; root:xnu-8796.141.3.703.2~2/RELEASE_ARM64_T6020

Steps to reproduce:

1. Have a SocketModeClient with these settings:

import os
from slack_sdk.socket_mode.websocket_client import SocketModeClient
from slack_sdk.web import WebClient

web_client = WebClient(token=os.environ["SLACK_WEB_TOKEN"])
socket_client = SocketModeClient(
    app_token=os.environ["SLACK_APP_TOKEN"],
    web_client=web_client,
    concurrency=100,
    auto_reconnect_enabled=True,
    trace_enabled=True,
)
socket_client.connect()

2. While that client is running, disconnect it from the Internet. This forces the active websockets to lose connection and simulates a DNS outage.
3. Wait until these stacktraces appear in an infinite loop:

2024-02-15 14:32:20,957 INFO:slack_sdk.socket_mode.websocket_client:Starting to receive messages from a new connection
2024-02-15 14:32:20,957 - slack_sdk.socket_mode.websocket_client - INFO - Starting to receive messages from a new connection
2024-02-15 14:32:20,958 ERROR:slack_sdk.socket_mode.websocket_client:Failed to start or stop the current session: socket is already opened
Traceback (most recent call last):
  File "/path/venv/lib/python3.9/site-packages/slack_sdk/socket_mode/websocket_client/__init__.py", line 237, in _run_current_session
    self.current_session.run_forever(
  File "/path/venv/lib/python3.9/site-packages/websocket/_app.py", line 422, in run_forever
    raise WebSocketException("socket is already opened")
websocket._exceptions.WebSocketException: socket is already opened

4. Reconnect the Internet and observe that the client never reconnects and continues providing the stacktraces above.

Expected result:

I expected the client to reconnect.

Actual result:

The client never reconnects.

Possible fix:

This function could set self.current_session = None in the event of an Exception.

python-slack-sdk/slack_sdk/socket_mode/websocket_client/__init__.py

Lines 233 to 246 in 60b02d6

	def _run_current_session(self):
	if self.current_session is not None:
	try:
	self.logger.info("Starting to receive messages from a new connection")
	self.current_session.run_forever(
	ping_interval=self.ping_interval,
	http_proxy_host=self.http_proxy_host,
	http_proxy_port=self.http_proxy_port,
	http_proxy_auth=self.http_proxy_auth,
	proxy_type=self.proxy_type,
	)
	self.logger.info("Stopped receiving messages from a connection")
	except Exception as e:
	self.logger.exception(f"Failed to start or stop the current session: {e}")

[seratch]

Thanks for reporting this issue. I'll look into this issue early next week.