balderdashy / balderdashy/sails

redis & redis-parser dependencies

Open
#4,411 3 comments 0 reactions 0 assignees View on GitHub
repro please
Dominant language
JavaScript
Stars
22.8k
Forks
1.9k
PR merge metrics
No merged PRs in 30d

Description

**Sails version**: 1.0.2
**Node version**: 10.1
**NPM version**: 6.0.1
**DB adapter name**: N/A
**DB adapter version**: N/A
**Operating system**: osx



Could you update **sails, connect-redis, socket.io-redis, sails-hook-sockets, machinepack-redis** to have latest redis & redis-parser in dependencies ?

redis-parser 2.6.0 has a [bug](https://github.com/NodeRedis/node-redis-parser/issues/46) and doesn't work well with high burst messages...

Contributor guide

Open the contributing guide

Research direction

Inspect the dependency manifests for sails, connect-redis, socket.io-redis, sails-hook-sockets, and machinepack-redis first. Check how their redis and redis-parser versions are resolved, then verify installation and high-burst message behavior; done means the affected packages use versions that address redis-parser 2.6.0's reported bug.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript, node.js, redis
Domain
backend
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.