Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cache documentation #2689

Closed
MaksimKiselev opened this issue Apr 3, 2023 · 1 comment
Closed

Cache documentation #2689

MaksimKiselev opened this issue Apr 3, 2023 · 1 comment

Comments

@MaksimKiselev
Copy link

In Caching section recommended to use cache-manager-redis-store but this unofficial store with totally incompatible implementation of CacheInterface dabroek/node-cache-manager-redis-store#53 (comment)

I've done research and find correct solution - use official redis/ioredis cache stores.

See: dabroek/node-cache-manager-redis-store#53 (comment)

I'm submitting a...

  • [+] Documentation issue or request (new chapter/page)

Current behavior

dabroek/node-cache-manager-redis-store#53 (comment)

Expected behavior

dabroek/node-cache-manager-redis-store#53 (comment)

Minimal reproduction of the problem with instructions

What is the motivation / use case for changing the behavior?

  1. Off docs should be useful
  2. Safe time for devs, because incorrectly implemented CacheInterface in cache-manager-redis-store package which also chages between versions it's evil and will broke app logic (dev - local cache store, prod - redis cache store === 💥 💔 ).

Environment

For Tooling issues:

  • Node version: XX
  • Platform:

Others:

@kamilmysliwiec
Copy link
Member

Let's track this here #2703

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants