`messageBuffer` support

Hi! 👋

Firstly, thanks for your work on this project! 🙂

Today I used patch-package to patch graphql-redis-subscriptions@2.3.1 for the project I’m working on.

Here is the diff that solved my problem:

diff --git a/node_modules/graphql-redis-subscriptions/dist/redis-pubsub.d.ts b/node_modules/graphql-redis-subscriptions/dist/redis-pubsub.d.ts
index bad3e89..d5a916c 100644
--- a/node_modules/graphql-redis-subscriptions/dist/redis-pubsub.d.ts
+++ b/node_modules/graphql-redis-subscriptions/dist/redis-pubsub.d.ts
@@ -37,5 +37,5 @@ export declare type Trigger = string | Path;
 export declare type TriggerTransform = (trigger: Trigger, channelOptions?: unknown) => string;
 export declare type Reviver = (key: any, value: any) => any;
 export declare type Serializer = (source: any) => string;
-export declare type Deserializer = (source: string) => any;
+export declare type Deserializer = (source: any) => any;
 export {};
diff --git a/node_modules/graphql-redis-subscriptions/dist/redis-pubsub.js b/node_modules/graphql-redis-subscriptions/dist/redis-pubsub.js
index 443893a..6ea54fa 100644
--- a/node_modules/graphql-redis-subscriptions/dist/redis-pubsub.js
+++ b/node_modules/graphql-redis-subscriptions/dist/redis-pubsub.js
@@ -82,8 +82,8 @@ var RedisPubSub = (function () {
                 console.error("No publisher or subscriber instances were provided and the package 'ioredis' wasn't found. Couldn't create Redis clients.");
             }
         }
-        this.redisSubscriber.on('pmessage', this.onMessage.bind(this));
-        this.redisSubscriber.on('message', this.onMessage.bind(this, undefined));
+        this.redisSubscriber.on('pmessageBuffer', this.onMessage.bind(this));
+        this.redisSubscriber.on('messageBuffer', this.onMessage.bind(this, undefined));
         this.subscriptionMap = {};
         this.subsRefsMap = {};
         this.currentSubscriptionId = 0;

This issue body was partially generated by patch-package.

[skabbes]

I’m not a maintainer, but I gave it a shot here: https://github.com/davidyaha/graphql-redis-subscriptions/pull/360

Selfishly just trying to breath a little life back into this project, so I can get some feedback on my issue too. https://github.com/davidyaha/graphql-redis-subscriptions/issues/359

Cheers, @AlaaZorkane hope this can get accepted.

[skabbes]

This definitely works, but I think it breaks backwards compatibility. Maybe an option(s) in the constructor to select “buffer mode” or not.

new RedisPubSub({
  ...
  messageEventName: 'messageBuffer',
  pmessageEventName: 'pmessageBuffer',
})

But that’s a little verbose, maybe:

new RedisPubSub({
  ...
  useBufferMessages: true,
})