This is an automated email from the ASF dual-hosted git repository.

scarb pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/rocketmq-clients.git


The following commit(s) were added to refs/heads/master by this push:
     new b4a47a93 [ISSUE #784] fix python client  (#783)
b4a47a93 is described below

commit b4a47a933dd1839d770b02bc7c385673a7ba9954
Author: yuz10 <845238...@qq.com>
AuthorDate: Thu Jul 4 23:04:02 2024 +0800

    [ISSUE #784] fix python client  (#783)
    
    fix python client
---
 python/rocketmq/signature.py                    |  6 +++++-
 python/rocketmq/simple_subscription_settings.py | 15 ++++-----------
 2 files changed, 9 insertions(+), 12 deletions(-)

diff --git a/python/rocketmq/signature.py b/python/rocketmq/signature.py
index 3b507ded..ed39f9b7 100644
--- a/python/rocketmq/signature.py
+++ b/python/rocketmq/signature.py
@@ -38,6 +38,10 @@ class Signature:
     __CREDENTIAL = "Credential"
     __SIGNED_HEADERS = "SignedHeaders"
     __SIGNATURE = "Signature"
+    try:
+        __CLIENT_VERSION = importlib.metadata.version("rocketmq")
+    except Exception:
+        __CLIENT_VERSION = "0.1.0"
 
     @staticmethod
     def sign(client_config: ClientConfig, client_id: str):
@@ -45,7 +49,7 @@ class Signature:
         metadata = [
             (Signature.__LANGUAGE_KEY, "PYTHON"),
             (Signature.__PROTOCOL_VERSION, "v2"),
-            (Signature.__CLIENT_VERSION_KEY, 
importlib.metadata.version("rocketmq")),
+            (Signature.__CLIENT_VERSION_KEY, Signature.__CLIENT_VERSION),
             (
                 Signature.__DATE_TIME_KEY,
                 date_time,
diff --git a/python/rocketmq/simple_subscription_settings.py 
b/python/rocketmq/simple_subscription_settings.py
index 6d193008..2ee60089 100644
--- a/python/rocketmq/simple_subscription_settings.py
+++ b/python/rocketmq/simple_subscription_settings.py
@@ -16,7 +16,7 @@
 from typing import Dict
 
 from google.protobuf.duration_pb2 import Duration
-from rocketmq.filter_expression import ExpressionType
+from rocketmq.filter_expression import ExpressionType, FilterExpression
 from rocketmq.log import logger
 from rocketmq.protocol.definition_pb2 import \
     FilterExpression as ProtoFilterExpression
@@ -30,13 +30,6 @@ from rocketmq.protocol.definition_pb2 import \
 from .settings import ClientType, ClientTypeHelper, Settings
 
 
-# Assuming a simple representation of FilterExpression for the purpose of this 
example
-class FilterExpression:
-    def __init__(self, type, expression):
-        self.Type = type
-        self.Expression = expression
-
-
 class SimpleSubscriptionSettings(Settings):
 
     def __init__(self, clientId, endpoints, consumerGroup, requestTimeout, 
longPollingTimeout,
@@ -60,12 +53,12 @@ class SimpleSubscriptionSettings(Settings):
             subscriptionEntry = ProtoSubscriptionEntry()
             filterExpression = ProtoFilterExpression()
 
-            if value.type == ExpressionType.Tag:
+            if value.type.value == ExpressionType.Tag.value:
                 filterExpression.type = ProtoFilterType.TAG
-            elif value.type == ExpressionType.Sql92:
+            elif value.type.value == ExpressionType.Sql92.value:
                 filterExpression.type = ProtoFilterType.SQL
             else:
-                logger.warn(f"[Bug] Unrecognized filter type={value.Type} for 
simple consumer")
+                logger.warn(f"[Bug] Unrecognized filter type={value.type} for 
simple consumer")
 
             filterExpression.expression = value.expression
             subscriptionEntry.topic.CopyFrom(topic)

Reply via email to