also fix selecting limits

0151669a · oroulet · oroulet · c810a2b8 · 0151669a
Commit 0151669a authored Sep 21, 2022 by oroulet Committed by oroulet Sep 21, 2022
Hide whitespace changes
Inline Side-by-side

Showing with 4 additions and 6 deletions

asyncua/common/connection.py asyncua/common/connection.py +4 -6

No files found.
--- a/asyncua/common/connection.py
+++ b/asyncua/common/connection.py
@@ -29,12 +29,10 @@ class TransportLimits:
    max_message_size: int = 100 * 1024 * 1024  # 100mb
    @staticmethod
-    def _select_limit(hint: ua.UInt32, limit: int) -> ua.UInt32:
+    def _select_limit(other: ua.UInt32, current_limit: int) -> ua.UInt32:
-        if limit <= 0:
+        if current_limit <= 0 or other <= 0:
-            return hint
+            return max(other, ua.UInt32(current_limit))
-        elif limit < hint:
+        return min(other, ua.UInt32(current_limit))
-            return hint
-        return ua.UInt32(limit)
    def is_msg_size_within_limit(self, sz: int) -> bool:
        if self.max_message_size == 0: