sycommon-python-lib 0.2.0b1__py3-none-any.whl → 0.2.0b2__py3-none-any.whl

sycommon/rabbitmq/rabbitmq_service_client_manager.py

@@ -28,62 +28,86 @@ class RabbitMQClientManager(RabbitMQCoreService):
 
     @classmethod
     async def _clean_client_resources(cls, client: RabbitMQClient) -> None:
-        """清理客户端无效资源（支持显式关闭独立通道）"""
+        """
+        清理客户端无效资源（修正版：只清理消费行为，不关闭 Channel）
+
+        重要变更：
+        - 移除了 await client._channel.close() 调用。
+        - 原因：关闭 Channel 可能会触发底层连接的 close_callback，导致重连死锁或冲突。
+        -       Channel 的关闭应该由 RabbitMQClient.connect() 内部的重建逻辑接管。
+        """
         try:
-            # 1. 先尝试正常停止消费
+            # 1. 尝试正常停止消费（发送 basic_cancel）
             if client._consumer_tag:
                 await client.stop_consuming()
         except Exception as e:
             logger.warning(f"停止消费逻辑异常: {str(e)}")
 
+        # 注意：这里不再显式关闭 client._channel
+        # 原因：
+        # 1. 如果是消费者独立通道，关闭它是安全的，但不如交给 connect() 统一处理（重建前关闭）。
+        # 2. 如果是生产者主通道（由 Pool 缓存），绝对不能在这里关闭！否则会影响其他生产者。
+        # 3. connect() 方法已经包含了 "显式关闭旧 Channel" 的逻辑，这里重复操作是多余的且危险的。
+
         try:
-            # 2. 【关键】显式关闭 Channel
-            # 在我们的新架构中，消费者持有独立通道，必须在这里显式关闭。
-            # 生产者持有的主通道虽然通常由 Pool 管理，但这里为了彻底清理，
-            # 可以尝试关闭（Pool 内部会有保护，或者接受主通道被关闭后重新创建）。
-            if client._channel and not client._channel.is_closed:
-                try:
-                    await client._channel.close()
-                    logger.debug(f"清理资源：已关闭客户端通道")
-                except Exception as e:
-                    logger.warning(f"清理资源：关闭通道异常: {e}")
-        except Exception as e:
-            logger.warning(f"清理资源过程发生异常: {str(e)}")
-        finally:
-            # 3. 强制重置客户端状态
-            # 注意：不要将 _channel_conn 置空，因为它是连接池的引用，置空后可能导致逻辑判断出错
-            # 但为了彻底断开，置空也是安全的，因为 connect 会重新获取。
+            # 2. 重置客户端状态引用（为后续 connect() 扫清障碍）
+            # 注意：虽然清理了状态，但不要清空 _message_handler！
             client._channel = None
             client._channel_conn = None
             client._exchange = None
             client._queue = None
             client._consumer_tag = None
+        except Exception as e:
+            logger.warning(f"重置客户端状态异常: {str(e)}")
 
     @classmethod
     async def _reconnect_client(cls, client_name: str, client: RabbitMQClient) -> bool:
-        """客户端重连"""
+        """客户端重连（修正版：保存与恢复 Handler，确保消费可恢复）"""
         if cls._is_shutdown or not (cls._connection_pool and await cls._connection_pool.is_alive):
             return False
 
         # 重连冷却
         await asyncio.sleep(cls.RECONNECT_INTERVAL)
 
+        # 【关键修复 1】状态快照
+        # 在清理资源前，保存关键状态，因为 _clean_client_resources 不会清空它，
+        # 但为了防御性编程，以及后续逻辑的清晰，我们显式保存。
+        saved_handler = client._message_handler
+        saved_queue_name = client.queue_name
+
         try:
-            # 清理旧资源
+            # 1. 清理旧资源
+            # 修正后的版本：只停止消费，不关闭 Channel
             await cls._clean_client_resources(client)
 
-            # 执行重连
+            # 2. 【关键修复 2】立即恢复 MessageHandler
+            # connect() 方法依赖 self._message_handler 来判断是否执行 "was_consuming" 的恢复逻辑。
+            # 如果不恢复，connect() 虽然会连上，但会跳过 start_consuming。
+            if saved_handler:
+                client._message_handler = saved_handler
+
+            # 防御性恢复队列名
+            if saved_queue_name:
+                client.queue_name = saved_queue_name
+
+            # 3. 执行重连
+            # 此时 client 处于“空资源但有 Handler”的状态，正是 connect() 期望的状态
             await client.connect()
 
             # 验证重连结果
             if await client.is_connected:
-                logger.info(f"客户端 '{client_name}' 重连成功")
+                logger.info(f"✅ 客户端 '{client_name}' 重连成功")
                 return True
             else:
-                logger.warning(f"客户端 '{client_name}' 重连失败：资源未完全初始化")
+                logger.warning(f"⚠️ 客户端 '{client_name}' 重连失败：资源未完全初始化")
                 return False
+
         except Exception as e:
-            logger.error(f"客户端 '{client_name}' 重连失败: {str(e)}", exc_info=True)
+            logger.error(
+                f"❌ 客户端 '{client_name}' 重连失败: {str(e)}", exc_info=True)
+            # 异常情况下也尝试恢复 handler，以便下次重试能成功
+            if saved_handler:
+                client._message_handler = saved_handler
             return False
 
     @classmethod

{sycommon_python_lib-0.2.0b1.dist-info → sycommon_python_lib-0.2.0b2.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: sycommon-python-lib
-Version: 0.2.0b1
+Version: 0.2.0b2
 Summary: Add your description here
 Requires-Python: >=3.11
 Description-Content-Type: text/markdown

{sycommon_python_lib-0.2.0b1.dist-info → sycommon_python_lib-0.2.0b2.dist-info}/RECORD

@@ -54,7 +54,7 @@ sycommon/notice/uvicorn_monitor.py,sha256=PrC-OFRE71mL8TcbfdkJRKbjwGAbgsWtyBPWIw
 sycommon/rabbitmq/rabbitmq_client.py,sha256=MC4QNSZpfYSPHY8-iW3RRuZAs0MZcNGWJD9EzCLc68k,22115
 sycommon/rabbitmq/rabbitmq_pool.py,sha256=igQ4Yh96oZyM8UEhNa_rljhZcitHwske3UsjPtBuj8c,18946
 sycommon/rabbitmq/rabbitmq_service.py,sha256=XSHo9HuIJ_lq-vizRh4xJVdZr_2zLqeLhot09qb0euA,2025
-sycommon/rabbitmq/rabbitmq_service_client_manager.py,sha256=O4zVL7lY2eaN_hUajwB5Ai6nnW-bevRBXysxSMhfVqw,9005
+sycommon/rabbitmq/rabbitmq_service_client_manager.py,sha256=NrJI4JKyItrMwQUFVZu0GWAx8krUdgUeyessFuUWhjo,10280
 sycommon/rabbitmq/rabbitmq_service_connection_monitor.py,sha256=uvoMuJDzJ9i63uVRq1NKFV10CvkbGnTMyEoq2rgjQx8,3013
 sycommon/rabbitmq/rabbitmq_service_consumer_manager.py,sha256=489r1RKd5WrTNMAcWCxUZpt9yWGrNunZlLCCp-M_rzM,11497
 sycommon/rabbitmq/rabbitmq_service_core.py,sha256=6RMvIf78DmEOZmN8dA0duA9oy4ieNswdGrOeyJdD6tU,4753
@@ -84,8 +84,8 @@ sycommon/tools/merge_headers.py,sha256=u9u8_1ZIuGIminWsw45YJ5qnsx9MB-Fot0VPge7it
 sycommon/tools/snowflake.py,sha256=xQlYXwYnI85kSJ1rZ89gMVBhzemP03xrMPVX9vVa3MY,9228
 sycommon/tools/syemail.py,sha256=BDFhgf7WDOQeTcjxJEQdu0dQhnHFPO_p3eI0-Ni3LhQ,5612
 sycommon/tools/timing.py,sha256=OiiE7P07lRoMzX9kzb8sZU9cDb0zNnqIlY5pWqHcnkY,2064
-sycommon_python_lib-0.2.0b1.dist-info/METADATA,sha256=5MhM3eTt-f3-du7nf8tlpH8wDKIU3_4z6nIRURL6Rg4,7372
-sycommon_python_lib-0.2.0b1.dist-info/WHEEL,sha256=wUyA8OaulRlbfwMtmQsvNngGrxQHAvkKcvRmdizlJi0,92
-sycommon_python_lib-0.2.0b1.dist-info/entry_points.txt,sha256=q_h2nbvhhmdnsOUZEIwpuoDjaNfBF9XqppDEmQn9d_A,46
-sycommon_python_lib-0.2.0b1.dist-info/top_level.txt,sha256=98CJ-cyM2WIKxLz-Pf0AitWLhJyrfXvyY8slwjTXNuc,17
-sycommon_python_lib-0.2.0b1.dist-info/RECORD,,
+sycommon_python_lib-0.2.0b2.dist-info/METADATA,sha256=YydQcYEWv2QGklHoJKu2z1mn8UT-Xtr8V_m0xW0Ig00,7372
+sycommon_python_lib-0.2.0b2.dist-info/WHEEL,sha256=wUyA8OaulRlbfwMtmQsvNngGrxQHAvkKcvRmdizlJi0,92
+sycommon_python_lib-0.2.0b2.dist-info/entry_points.txt,sha256=q_h2nbvhhmdnsOUZEIwpuoDjaNfBF9XqppDEmQn9d_A,46
+sycommon_python_lib-0.2.0b2.dist-info/top_level.txt,sha256=98CJ-cyM2WIKxLz-Pf0AitWLhJyrfXvyY8slwjTXNuc,17
+sycommon_python_lib-0.2.0b2.dist-info/RECORD,,