dragonfly icon indicating copy to clipboard operation
dragonfly copied to clipboard

messageBuffer, pmessageBuffer

Open yovanoc opened this issue 3 years ago • 9 comments

Hello, I'm using IORedis, and I would like to use "buffer" hooks, thanks

yovanoc avatar Oct 12 '22 06:10 yovanoc

Could you expand? Which commands are you referring to? Alternatively, a code snippet that reproduces the problem will help.

romange avatar Oct 12 '22 06:10 romange

I don't know much more but here's from the ioredis readme: (https://github.com/luin/ioredis)

image

We can listen buffer instead of string, and if I switch to dragonfly instead of a classic redis image it stop working

yovanoc avatar Oct 12 '22 07:10 yovanoc

Did you try using "message" instead of "messageBuffer"?

romange avatar Oct 12 '22 07:10 romange

Yes, message work and give me strings, I want buffer. That's why they added messageBuffer

yovanoc avatar Oct 12 '22 07:10 yovanoc

@yovanoc Thanks! I am just trying to pinpoint the issue here :)

I opened a task for this. Someone in the community will surely help me with setting up a nodejs test that reproduces the problem and I will be able to fix it then.

romange avatar Oct 12 '22 08:10 romange

Perfect, I will read this and see if I can help too ! thanks !

yovanoc avatar Oct 12 '22 08:10 yovanoc

@yovanoc we can not reproduce this on our side. We run tests from https://github.com/luin/ioredis/blob/main/test/functional/pub_sub.ts and they pass. Can you provide a minimal example that demonstrates this issue?

romange avatar Oct 19 '22 05:10 romange

Ok weird, I will try to do a minimal reproduction then, thanks

yovanoc avatar Oct 19 '22 06:10 yovanoc

We did not succeed to reproduce. Will close after chrismukkah unless a minimal example is provided.

romange avatar Nov 09 '22 09:11 romange

Closing - unable to reproduce

romange avatar Jan 07 '23 06:01 romange