Fix issue with pack_commands returning an empty byte sequence

redis · Jan 25, 2023 · fa669ff · fa669ff
1 parent e39c7ba
commit fa669ff
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/redis/asyncio/connection.py b/redis/asyncio/connection.py
@@ -922,7 +922,8 @@ def pack_commands(self, commands: Iterable[Iterable[EncodableT]]) -> List[bytes]
                     or chunklen > buffer_cutoff
                     or isinstance(chunk, memoryview)
                 ):
-                    output.append(SYM_EMPTY.join(pieces))
+                    if pieces:
+                        output.append(SYM_EMPTY.join(pieces))
                     buffer_length = 0
                     pieces = []
 

diff --git a/redis/connection.py b/redis/connection.py
@@ -930,7 +930,8 @@ def pack_commands(self, commands):
                     or chunklen > buffer_cutoff
                     or isinstance(chunk, memoryview)
                 ):
-                    output.append(SYM_EMPTY.join(pieces))
+                    if pieces:
+                        output.append(SYM_EMPTY.join(pieces))
                     buffer_length = 0
                     pieces = []