webex-bot 1.0.2__tar.gz → 1.0.4__tar.gz

{webex_bot-1.0.2/webex_bot.egg-info → webex_bot-1.0.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: webex_bot
-Version: 1.0.2
+Version: 1.0.4
 Summary: Python package for a Webex Bot based on websockets.
 Home-page: https://github.com/fbradyirl/webex_bot
 Author: Finbarr Brady
@@ -447,9 +447,15 @@ bot = WebexBot(teams_bot_token=os.getenv("WEBEX_ACCESS_TOKEN")
 
 * Fix for [issue #48][i48] - Fix for `Commands not being received` issue.
 
-### 1.0.2 (2025-Jun-04)
+### 1.0.3 (2025-Jun-04)
 
 * Add connection headers to requests.
+* Only call me people API once per run.
+
+### 1.0.4 (2025-Jul-01)
+
+* Add retry mechanism with backoff for websocket 404 errors
+*
 
 [1]: https://github.com/aaugustin/websockets
 

{webex_bot-1.0.2 → webex_bot-1.0.4}/README.md

@@ -407,9 +407,15 @@ bot = WebexBot(teams_bot_token=os.getenv("WEBEX_ACCESS_TOKEN")
 
 * Fix for [issue #48][i48] - Fix for `Commands not being received` issue.
 
-### 1.0.2 (2025-Jun-04)
+### 1.0.3 (2025-Jun-04)
 
 * Add connection headers to requests.
+* Only call me people API once per run.
+
+### 1.0.4 (2025-Jul-01)
+
+* Add retry mechanism with backoff for websocket 404 errors
+*
 
 [1]: https://github.com/aaugustin/websockets
 

{webex_bot-1.0.2 → webex_bot-1.0.4}/setup.cfg

@@ -1,5 +1,5 @@
 [bumpversion]
-current_version = 1.0.2
+current_version = 1.0.4
 commit = True
 tag = True
 

{webex_bot-1.0.2 → webex_bot-1.0.4}/setup.py

@@ -45,6 +45,6 @@ setup(
     test_suite='tests',
     tests_require=test_requirements,
     url='https://github.com/fbradyirl/webex_bot',
-    version='1.0.2',
+    version='1.0.4',
     zip_safe=False,
 )

{webex_bot-1.0.2 → webex_bot-1.0.4}/webex_bot/__init__.py

@@ -1,4 +1,4 @@
 """Top-level package for Webex Bot."""
 
 __author__ = """Finbarr Brady"""
-__version__ = '1.0.2'
+__version__ = '1.0.4'

{webex_bot-1.0.2 → webex_bot-1.0.4}/webex_bot/webex_bot.py

@@ -23,9 +23,9 @@ class WebexBot(WebexWebsocketClient):
 
     def __init__(self,
                  teams_bot_token,
-                 approved_users=[],
-                 approved_domains=[],
-                 approved_rooms=[],
+                 approved_users=None,
+                 approved_domains=None,
+                 approved_rooms=None,
                  include_demo_commands=False,
                  bot_name="Webex Bot",
                  bot_help_subtitle="Here are my available commands. Click one to begin.",
@@ -61,11 +61,12 @@ class WebexBot(WebexWebsocketClient):
                                       on_card_action=self.process_incoming_card_action,
                                       proxies=proxies)
 
+        me = self.get_me_info()
         if help_command is None:
             self.help_command = HelpCommand(
                 bot_name=bot_name,
                 bot_help_subtitle=bot_help_subtitle,
-                bot_help_image=self.teams.people.me().avatar)
+                bot_help_image=me.avatar)
         else:
             self.help_command = help_command
 
@@ -83,12 +84,11 @@ class WebexBot(WebexWebsocketClient):
         self.help_command.commands = self.commands
 
         self.card_callback_commands = {}
-        self.approved_users = approved_users
-        self.approved_domains = approved_domains
-        self.approved_rooms = approved_rooms
+        self.approved_users = approved_users if approved_users is not None else []
+        self.approved_domains = approved_domains if approved_domains is not None else []
+        self.approved_rooms = approved_rooms if approved_rooms is not None else []
         self.approval_parameters_check()
         self.bot_display_name = ""
-        self.get_me_info()
         self.threads = threads
 
     @backoff.on_exception(backoff.expo, requests.exceptions.ConnectionError)
@@ -100,6 +100,7 @@ class WebexBot(WebexWebsocketClient):
         self.bot_display_name = me.displayName
         log.info(f"Running as {me.type} '{me.displayName}' with email {me.emails}")
         log.debug(f"Running as bot '{me}'")
+        return me
 
     def add_command(self, command_class: Command):
         """

{webex_bot-1.0.2 → webex_bot-1.0.4}/webex_bot/websockets/webex_websocket_client.py

@@ -276,22 +276,47 @@ class WebexWebsocketClient(object):
                 while True:
                     await _websocket_recv()
 
-        try:
-            asyncio.get_event_loop().run_until_complete(_connect_and_listen())
-        except InvalidStatusCode as e:
-            logger.error(f"WebSocket handshake to {ws_url} failed with status {e.status_code}")
-            if e.status_code == 404:
-                logger.info("Refreshing WDM device info and retrying...")
-                self._get_device_info(check_existing=False)
-                # update ws_url before retry
-                ws_url = self.device_info.get('webSocketUrl')
+        # Track the number of consecutive 404 errors to prevent infinite loops
+        max_404_retries = 3
+        current_404_retries = 0
+        
+        while True:
+            try:
                 asyncio.get_event_loop().run_until_complete(_connect_and_listen())
-            else:
-                raise
-        except Exception as runException:
-            logger.error(f"runException: {runException}")
-            if self._get_device_info(check_existing=False) is None:
-                logger.error('could not create device info')
-                raise Exception("No WDM device info")
-            # trigger re-connect
-            asyncio.get_event_loop().run_until_complete(_connect_and_listen())
+                # If we get here, the connection was successful, so break out of the loop
+                break
+            except InvalidStatusCode as e:
+                logger.error(f"WebSocket handshake to {ws_url} failed with status {e.status_code}")
+                
+                if e.status_code == 404:
+                    current_404_retries += 1
+                    if current_404_retries >= max_404_retries:
+                        logger.error(f"Reached maximum retries ({max_404_retries}) for 404 errors. Giving up.")
+                        raise Exception(f"Unable to connect to WebSocket after {max_404_retries} attempts. Device registration may be invalid.")
+                    
+                    logger.info(f"Refreshing WDM device info and retrying... (Attempt {current_404_retries} of {max_404_retries})")
+                    # Force a new device registration
+                    self._get_device_info(check_existing=False)
+                    # Update ws_url with the new device info
+                    ws_url = self.device_info.get('webSocketUrl')
+                    
+                    # Add a delay before retrying to avoid hammering the server
+                    logger.info(f"Waiting 5 seconds before retry attempt {current_404_retries}...")
+                    asyncio.get_event_loop().run_until_complete(asyncio.sleep(5))
+                else:
+                    # For non-404 errors, just raise the exception
+                    raise
+            except Exception as runException:
+                logger.error(f"runException: {runException}")
+                
+                # Check if we can get device info
+                if self._get_device_info(check_existing=False) is None:
+                    logger.error('could not create device info')
+                    raise Exception("No WDM device info")
+                    
+                # Update the URL in case it changed
+                ws_url = self.device_info.get('webSocketUrl')
+                
+                # Wait a bit before reconnecting
+                logger.info("Waiting 5 seconds before attempting to reconnect...")
+                asyncio.get_event_loop().run_until_complete(asyncio.sleep(5))

{webex_bot-1.0.2 → webex_bot-1.0.4/webex_bot.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: webex_bot
-Version: 1.0.2
+Version: 1.0.4
 Summary: Python package for a Webex Bot based on websockets.
 Home-page: https://github.com/fbradyirl/webex_bot
 Author: Finbarr Brady
@@ -447,9 +447,15 @@ bot = WebexBot(teams_bot_token=os.getenv("WEBEX_ACCESS_TOKEN")
 
 * Fix for [issue #48][i48] - Fix for `Commands not being received` issue.
 
-### 1.0.2 (2025-Jun-04)
+### 1.0.3 (2025-Jun-04)
 
 * Add connection headers to requests.
+* Only call me people API once per run.
+
+### 1.0.4 (2025-Jul-01)
+
+* Add retry mechanism with backoff for websocket 404 errors
+*
 
 [1]: https://github.com/aaugustin/websockets