feathersjs-ecosystem / feathers-sync

Synchronize service events between Feathers application instances
MIT License
221 stars 41 forks source link

fix: Do not require uri if redisClient object provided. #196

Open palmtown opened 4 months ago

palmtown commented 4 months ago

Summary

If passing a redisClient object as shown in Exhibit 1, current code will throw an error as it is expecting the uri. However, when passing the redisClient the uri is not required.

Exhibit 1 app.configure( sync({ redisClient }) );

https://github.com/feathersjs-ecosystem/feathers-sync/issues/197

No

If so, please mention them to keep the conversations linked together.

Other Information