Skip Menu |

This queue is for tickets about the Protocol-Redis-XS CPAN distribution.

Report information
The Basics
Id: 128365
Status: resolved
Priority: 0/
Queue: Protocol-Redis-XS

People
Owner: Nobody in particular
Requestors: alex.karelas [...] gmail.com
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: 0.06



Subject: Add support for nested multi bulk replies with depth > 2
Date: Sat, 26 Jan 2019 14:03:01 +0200
To: bug-Protocol-Redis-XS [...] rt.cpan.org
From: Alexander Karelas <alex.karelas [...] gmail.com>
jhthorsen says: I got a bug report in my project Mojo::Redis, which I cannot fix. It's regarding reading the response from the stream <https://redis.io/topics/streams-intro> commands. the problem is that the nesting is higher than what is supported by this module. I'm not even sure why you have a max depth "guard" and why it is so low. Please see jhthorsen/mojo-redis#34 <https://github.com/jhthorsen/mojo-redis/issues/34> for more details. https://redis.io/topics/streams-intro https://github.com/jhthorsen/mojo-redis/issues/34 https://github.com/dgl/protocol-redis-xs/issues/5
On Sat Jan 26 07:03:52 2019, alex.karelas@gmail.com wrote: Show quoted text
> jhthorsen says: > > I got a bug report in my project Mojo::Redis, which I cannot fix. > > It's regarding reading the response from the stream > <https://redis.io/topics/streams-intro> commands. the problem is that > the nesting is higher than what is supported by this module. > > I'm not even sure why you have a max depth "guard" and why it is so low. > > Please see jhthorsen/mojo-redis#34 > <https://github.com/jhthorsen/mojo-redis/issues/34> for more details. > > https://redis.io/topics/streams-intro > > https://github.com/jhthorsen/mojo-redis/issues/34 > > https://github.com/dgl/protocol-redis-xs/issues/5 >
This should be resolved in 0.06 by requiring or installing a newer version of hiredis that supports up to 7 levels of nesting.