[blocking] Update node-redis to 4, rewrite connect-redis #1

Closed
opened 2022-06-17 20:53:34 +00:00 by hf · 0 comments
Owner

connect-redis needs legacyMode enabled in node-redis to work with version >4, which comes with many own problems.

Since it's only a 200 lines file, it is better to include our own and rewrite it for our needs, one user already did some work here.

Documentation on how to write express connectors is here

Any other update doesn't make sense if we do not have a stable use of redis.

[connect-redis](https://github.com/tj/connect-redis/blob/master/lib/connect-redis.js) needs legacyMode enabled in node-redis to work with version >4, which comes with many own problems. Since it's only a 200 lines file, it is better to include our own and rewrite it for our needs, one user already did some work [here](https://github.com/BarryCarlyon/connect-redis/blob/supportv4/lib/connect-redis.js). Documentation on how to write express connectors is [here](https://github.com/expressjs/session) Any other update doesn't make sense if we do not have a stable use of redis.
hf referenced this issue from a commit 2022-06-19 15:30:18 +00:00
hf closed this issue 2022-06-19 15:30:18 +00:00
Sign in to join this conversation.
No Milestone
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: ppfun/pixelplanet#1
No description provided.