Upgrade redis to v4 #40

dantehemerson · 2022-04-20T14:07:55Z

To use newest types features for typescript and avoid errors with types when using latest 3.x versions

sarcastic-verma · 2022-06-28T22:06:44Z

Any update on when will this be completed?

vxm5091 · 2022-06-30T02:09:00Z

+1

Sravanchittimalla · 2022-07-15T16:42:19Z

+1

celianvdb · 2022-07-18T07:57:19Z

+1

dabroek · 2022-07-18T12:12:21Z

This is in the works. Stay tuned.

FoxxMD · 2022-09-01T18:17:39Z

Hi, has there been any progress on this?

muratdemirci · 2022-09-05T12:41:29Z

+1

tvacherat · 2022-09-26T10:09:59Z

+1

andreaziz · 2022-10-11T14:35:01Z

+1

smentek · 2022-10-12T18:12:40Z

+1

alibeknow · 2022-10-13T05:06:31Z

+1

mehamednews · 2022-10-14T05:55:14Z

+1

dabroek · 2022-10-15T01:10:05Z

First of all, thank you all for showing your interest. It encouraged me to finally work on this again.

I am happy to inform you that I have just released v3.0.0 of this package on npm which upgrades the redis dependency to redis@^4.3.1 and adds new TypeScript declarations.

Please let me know if you have any feedback or further requests.

DmytroHaponovMD · 2022-10-17T11:09:09Z

Is it work now for Nest.js ?

dabroek · 2022-10-17T19:59:32Z

@DmytroHaponovMD This can currently be achieved using:

import { redisStore } from 'cache-manager-redis-store';
import { CacheModule, Module } from '@nestjs/common';
import { AppController } from './app.controller';
import { AppService } from './app.service';

@Module({
  imports: [
    CacheModule.register({
      // @ts-ignore
      store: async () => await redisStore({
        // Store-specific configuration:
        socket: {
          host: 'localhost',
          port: 6379,
        }
      })
    }),
  ],
  controllers: [AppController],
  providers: [AppService],
})
export class AppModule {}

Unfortunately you are going to need the // @ts-ignore for now, since the interface does not allow Promise<CacheStore> to be returned.

NarHakobyan · 2022-10-20T11:59:36Z

@DmytroHaponovMD This can currently be achieved using:

import { redisStore } from 'cache-manager-redis-store';
import { CacheModule, Module } from '@nestjs/common';
import { AppController } from './app.controller';
import { AppService } from './app.service';

@Module({
  imports: [
    CacheModule.register({
      // @ts-ignore
      store: async () => await redisStore({
        // Store-specific configuration:
        socket: {
          host: 'localhost',
          port: 6379,
        }
      })
    }),
  ],
  controllers: [AppController],
  providers: [AppService],
})
export class AppModule {}

Unfortunately you are going to need the // @ts-ignore for now, since the interface does not allow Promise<CacheStore> to be returned.

this can't work becouse store field is required for option passed in redisStore function

smentek · 2022-10-20T12:12:49Z

@DmytroHaponovMD This can currently be achieved using:

import { redisStore } from 'cache-manager-redis-store';
import { CacheModule, Module } from '@nestjs/common';
import { AppController } from './app.controller';
import { AppService } from './app.service';

@Module({
  imports: [
    CacheModule.register({
      // @ts-ignore
      store: async () => await redisStore({
        // Store-specific configuration:
        socket: {
          host: 'localhost',
          port: 6379,
        }
      })
    }),
  ],
  controllers: [AppController],
  providers: [AppService],
})
export class AppModule {}

Unfortunately you are going to need the // @ts-ignore for now, since the interface does not allow Promise<CacheStore> to be returned.

this can't work becouse store field is required for option passed in redisStore function

Is it not why // @ts-ignore is there?

NarHakobyan · 2022-10-20T12:29:55Z

@DmytroHaponovMD This can currently be achieved using:

import { redisStore } from 'cache-manager-redis-store';
import { CacheModule, Module } from '@nestjs/common';
import { AppController } from './app.controller';
import { AppService } from './app.service';

@Module({
  imports: [
    CacheModule.register({
      // @ts-ignore
      store: async () => await redisStore({
        // Store-specific configuration:
        socket: {
          host: 'localhost',
          port: 6379,
        }
      })
    }),
  ],
  controllers: [AppController],
  providers: [AppService],
})
export class AppModule {}

Unfortunately you are going to need the // @ts-ignore for now, since the interface does not allow Promise<CacheStore> to be returned.

this can't work becouse store field is required for option passed in redisStore function

Is it not why // @ts-ignore is there?

// @ts-ignore works for next line, so effected line is store: async () => await redisStore({, but error is inside redisStore, there is another store property sibling of socket.

FoxxMD · 2022-10-20T12:56:05Z

@dabroek The DefinitelyTyped typings for this package need to be updated for ^3.0 https://www.npmjs.com/package/@types/cache-manager-redis-store -- this is the root of the error @DmytroHaponovMD is seeing, I believe

smentek · 2022-10-20T13:06:12Z

This seems to be working for me:

@Module({
  imports: [
    ['true', '1', 'yes'].includes(<string>process.env.API_REDIS_STORE_IS_ACTIVE)
      ? CacheModule.register({
          isGlobal: true,
          // eslint-disable-next-line @typescript-eslint/ban-ts-comment
          // @ts-ignore
          store: async () => {
            return await redisStore({
              // Store-specific configuration:
              socket: {
                host: process.env.API_REDIS_HOST,
                port: +(<string>process.env.API_REDIS_PORT),
              },
            });
          },
        })
      : CacheModule.register({ isGlobal: true }),
...

With

"cache-manager": "^5.1.1",
"cache-manager-redis-store": "^3.0.1",

assisgui · 2022-10-20T19:13:17Z

I found a workaround to use registerAsync:

import { CacheStore } from '@nestjs/common/cache/interfaces/cache-manager.interface'

CacheModule.registerAsync({
    imports: [ConfigModule],
    useFactory: async (config: ConfigService) =>
      await redisStore({
        socket: {
          host: config.get('REDIS_HOST'),
          port: +config.get('REDIS_PORT'),
        },
      }) as unknown as CacheStore,
    inject: [ConfigService],
  }),

Also:

"cache-manager": "^5.1.1",
"cache-manager-redis-store": "^3.0.1",

cskiwi · 2022-10-21T14:14:41Z

@assisgui same here but had to also configure the ttl:

CacheModule.registerAsync({
  imports: [ConfigModule],
  useFactory: async (config: ConfigService) => {
    const store = await redisStore({
      socket: {
        host: config.get('REDIS_HOST'),
        port: +config.get('REDIS_PORT'),
      },
      password: config.get('REDIS_PASSWORD'),
    });

    return {
      store: store as unknown as CacheStore,
      ttl: 60 * 60 * 24 * 7,
    };
  },
  inject: [ConfigService],
}),

SerhiiBabich · 2022-10-22T10:38:32Z

Any update on when will this be completed?

mihailgolban · 2022-10-27T12:01:20Z

Using cache-manager '5.1.1' I was getting the following error:

 TypeError: store.get is not a function

It worked for me after the cache-manager downgrade version to 4.1.0

@Module({
  imports: [
    CacheModule.registerAsync({
      isGlobal: true,
      inject: [ConfigService],
      useFactory: async (configService: ConfigService) => {
        const store = await redisStore({
          socket: {
            host: config.get('REDIS_HOST'),
            port: +config.get('REDIS_PORT'),
          },
          password: config.get('REDIS_PASSWORD'),
        });
        return {
          store: store as unknown as CacheStore,
          ttl: 5
        };
      }
    })
  ]
})
export class RedisModule {}

"@nestjs/core": "9.0.11",
"cache-manager": "4.1.0",
"cache-manager-redis-store": "3.0.1",

SerhiiBabich · 2022-10-27T14:09:03Z

Using cache-manager '5.1.1' I was getting the following error:

 TypeError: store.get is not a function

It worked for me after the cache-manager downgrade version to 4.1.0

@Module({
  imports: [
    CacheModule.registerAsync({
      isGlobal: true,
      inject: [ConfigService],
      useFactory: async (configService: ConfigService) => {
        const store = await redisStore({
          socket: {
            host: config.get('REDIS_HOST'),
            port: +config.get('REDIS_PORT'),
          },
          password: config.get('REDIS_PASSWORD'),
        });
        return {
          store: store as unknown as CacheStore,
          ttl: 5
        };
      }
    })
  ]
})
export class RedisModule {}

"@nestjs/core": "9.0.11",
"cache-manager": "4.1.0",
"cache-manager-redis-store": "3.0.1",

In my case - it doesn't work, and I do not want to check all time is here fix or not.
So, the question is still actual:

Any update on when will this be completed?

NarHakobyan · 2022-10-27T20:28:16Z

CacheModule.registerAsync<RedisClientOptions>({
      isGlobal: true,
      imports: [SharedModule],
      inject: [ApiConfigService],
      useFactory: async (configService: ConfigService) => {
        const store = await redisStore({
          socket: {
            host: config.get('REDIS_HOST'),
            port: +config.get('REDIS_PORT'),
          },
          password: config.get('REDIS_PASSWORD'),
        });

        return {
          store: {
          	create: () => store as unknown as CacheStore,
          },
          ttl: 60 * 60 * 24 * 7, // 1 week
        };
      },
    }),

works fine for me, just copy and use above code :)

conioX · 2022-11-03T00:42:41Z

Hi @dabroek, any news for the Type, my repo not allow ts ignore lol

kamal-brill · 2022-11-23T13:23:15Z

Can anyone please share the updated, working code with the modules & versions ?

And also please help on how to connect to redis cluster with auth ? (AWS ElastiCache)

codeprefect · 2022-11-27T17:22:48Z

I have this, and it works

@Module({
  imports: [
    ConfigModule,
    CacheModule.registerAsync<any>({
      isGlobal: true,
      imports: [ConfigModule],
      useFactory: async (configService: ConfigService) => {
        const options = await configService.getRedisOptions();
        const store = await redisStore({
          socket: { host: options.host, port: options.port },
          ttl: 60,
        });
        return {
          store: () => store,
        };
      },
      inject: [ConfigService],
    }),  
  ],
  exports: [ConfigModule],
})
export class CommonModule {}

kamal-brill · 2022-11-28T12:02:34Z

@codeprefect Could you please share the below dependencies versions from your package.json ?

"cache-manager"
"cache-manager-redis-store"
"redis"
"@nestjs/core"

And also please let me know if the above code works for AWS ElastiCache CLUSTER with authorization, because the above code doesn't specify any of that.

codeprefect · 2022-11-29T09:28:44Z

Hey @kamal-brill ,
This should work for you.

const store = await redisStore({
      socket: { host: <your host>, port: <your port> },
      username: <username, usually (default)>,
      password: <password>,
      database: <redis database (0 - 15)>,
})

I tested it with a redislabs instance.

codeprefect · 2022-11-29T09:30:06Z

One regretful thing is that the ttl option is not applied when using cache manager's set method by injecting cache-manager in *.service.ts. Fortunately, the @CacheTTL() decorate is working.

It seems like set method expects ttl option as an object value not a number, which is misled by type definition.

node-cache-manager-redis-store/index.js

Line 19 in 04d04cb

const ttl = (options?.ttl || options?.ttl === 0) ? options.ttl : config.ttl;

It works after I changed like below.
this.cacheManager.set("key", "value", { ttl: 10 } as any);
version
"cache-manager": "^5.1.3",
"cache-manager-redis-store": "^3.0.1",

I have also confirmed this issue.

zbingsong · 2023-01-14T07:40:15Z

One regretful thing is that the ttl option is not applied when using cache manager's set method by injecting cache-manager in *.service.ts. Fortunately, the @CacheTTL() decorate is working.

It seems like set method expects ttl option as an object value not a number, which is misled by type definition.

node-cache-manager-redis-store/index.js

Line 19 in 04d04cb

const ttl = (options?.ttl || options?.ttl === 0) ? options.ttl : config.ttl;

It works after I changed like below.
this.cacheManager.set("key", "value", { ttl: 10 } as any);
version
"cache-manager": "^5.1.3",
"cache-manager-redis-store": "^3.0.1",

This is the only solution that worked for me.

wodCZ · 2023-01-15T16:27:42Z

Since the issue is still active, and there's no clear solution posted, allow me to provide a clean fix:

Author of the original node-cache-manager library has forked this client and adapted the required fixes. It is compatible with latest versions of cache-manager and redis libraries. Replace this library with cache-manager-redis-yet and you're good to go.

The rest of the comment is only relevant for NestJS users.

How to use redis with NestJS cache manager

Install cache-manager-redis-yet using your package manager.

import { CacheModule, Module } from '@nestjs/common';
import { AppController } from './app.controller';
import { AppService } from './app.service';
import { redisStore } from 'cache-manager-redis-yet';

@Module({
  imports: [
    CacheModule.registerAsync({
      useFactory: async () => ({
        store: await redisStore({ ttl: 5000 }),
      }),
    }),
  ],
  controllers: [AppController],
  providers: [AppService],
})
export class AppModule {}

You can extend on this to provide the required connection credentials to await redisStore() from config, just follow NestJS docs.

Notes about TTL

cache-manager v4 was using seconds for TTL, cache-manager v5 has switched to milliseconds. Use milliseconds where NestJS docs say seconds.
you have to provide your default ttl to await redisStore({ ttl: 5000 }). Setting ttl next to store doesn't work. Also, NestJS' default 5s TTL doesn't apply when using redisStore — provide it manually as illustrated.
Overriding global TTL using @CacheTTL(x) doesn't currently work, because of another change in cache-manager 5. I've provided a workaround that you can use until this is fixed on NestJS side. Same thing applies, use milliseconds instead of seconds with CacheTTL.

Warning
Make sure you're using NestJS v 9.2+, since there was another issue with cache-manager 5 that was recently fixed.

Note
If you found this helpful, please add reaction instead of writing a new comment, so this doesn't get lost in the thread.
Thanks 🙏

puzzle9 · 2023-01-26T16:04:39Z

Since the issue is still active, and there's no clear solution posted, allow me to provide a clean fix:

Author of the original node-cache-manager library has forked this client and adapted the required fixes. It is compatible with latest versions of cache-manager and redis libraries. Replace this library with cache-manager-redis-yet and you're good to go.

The rest of the comment is only relevant for NestJS users.

How to use redis with NestJS cache manager

Install cache-manager-redis-yet using your package manager.
import { CacheModule, Module } from '@nestjs/common';
import { AppController } from './app.controller';
import { AppService } from './app.service';
import { redisStore } from 'cache-manager-redis-yet';

@Module({
  imports: [
    CacheModule.registerAsync({
      useFactory: async () => ({
        store: await redisStore({ ttl: 5000 }),
      }),
    }),
  ],
  controllers: [AppController],
  providers: [AppService],
})
export class AppModule {}
You can extend on this to provide the required connection credentials to await redisStore() from config, just follow NestJS docs.

Notes about TTL

cache-manager v4 was using seconds for TTL, cache-manager v5 has switched to milliseconds. Use milliseconds where NestJS docs say seconds.

you have to provide your default ttl to await redisStore({ ttl: 5000 }). Setting ttl next to store doesn't work. Also, NestJS' default 5s TTL doesn't apply when using redisStore — provide it manually as illustrated.

Overriding global TTL using @CacheTTL(x) doesn't currently work, because of another change in cache-manager 5. I've provided a workaround that you can use until this is fixed on NestJS side. Same thing applies, use milliseconds instead of seconds with CacheTTL.

Warning
Make sure you're using NestJS v 9.2+, since there was another issue with cache-manager 5 that was recently fixed.

Note
If you found this helpful, please add reaction instead of writing a new comment, so this doesn't get lost in the thread.
Thanks pray

to change

import { CacheModule, Module } from '@nestjs/common';
import { AppController } from './app.controller';
import { AppService } from './app.service';
import { redisStore } from 'cache-manager-redis-yet';

@Module({
  imports: [
    CacheModule.registerAsync({
      useFactory: async () => await redisStore({ ttl: 5000 }),
    }),
  ],
  controllers: [AppController],
  providers: [AppService],
})
export class AppModule {}

to fix

this.cacheManager.get(key) return store.get is not a function

ajfranzoia · 2023-03-20T20:48:05Z

For me it worked with cache-manager-redis-yet using this config:

useFactory: async () => {
  return {
    store: await redisStore({
      // client options
    }),
  };
}

anilahir · 2023-04-03T12:24:23Z

Following async configuration working fine for me.

CacheModule.registerAsync<RedisClientOptions>({
  imports: [ConfigModule],
  inject: [ConfigService],
  useFactory: async (configService: ConfigService) => ({
    store: redisStore as unknown as CacheStore,
    url: `redis://${configService.get<string>('REDIS_HOST')}:${configService.get<number>('REDIS_PORT')}`,
    database: configService.get<number>('REDIS_DB'),
    ttl: 24 * 60 * 60, // 1 day
  }),
  isGlobal: true,
})

Justin3go · 2023-05-14T08:00:29Z

我想要在docker中使用，但会一直显示redis的自定义配置无效的问题，就像下面这样，无论port还是host都仍然使用的默认的：

我是这样解决的，在redisStore中配置，可以成功，但没有在任何文档中找到这样的配置，不知道是否符合实践：

    CacheModule.registerAsync<RedisClientOptions>({
      isGlobal: true,
      imports: [ConfigModule],
      inject: [ConfigService],
      useFactory: async (configService: ConfigService) => ({
        store: await redisStore({
          ttl: 5000,
          url: `redis://${configService.get<string>(
            'REDIS_HOST'
          )}:${configService.get<number>('REDIS_PORT')}`,
        }),
      }),
    }),

Akupsmee · 2023-06-02T07:09:56Z

this works for me with the latest package versions:

CacheModule.register({
      store: async () =>
        await redisStore({
          socket: {
            host: 'localhost',
            port: 6379,
          },
          ttl: 10000,
        }),
      isGlobal: true,
    })

igmrrf · 2023-06-03T13:41:48Z

@assisgui same here but had to also configure the ttl:

CacheModule.registerAsync({
  imports: [ConfigModule],
  useFactory: async (config: ConfigService) => {
    const store = await redisStore({
      socket: {
        host: config.get('REDIS_HOST'),
        port: +config.get('REDIS_PORT'),
      },
      password: config.get('REDIS_PASSWORD'),
    });

    return {
      store: store as unknown as CacheStore,
      ttl: 60 * 60 * 24 * 7,
    };
  },
  inject: [ConfigService],
}),

This seems to work with latest packages, the former left the TTL to -1. Noticed cached items didn't expire and checked the database.

CacheModule.registerAsync({
    imports: [ConfigModule],
    useFactory: async (config: ConfigService) => {
        const store = await redisStore({
            socket: {
                host: config.get<string>('REDIS_HOST'),
                port: config.get<number>('REDIS_PORT'),
            },
            password: config.get<string>('REDIS_PASSWORD'),
            ttl: 60, // 60 seconds
        });

        return {
            store: store as unknown as CacheStore,
        };
    },
    inject: [ConfigService],
}),

 "@nestjs/cache-manager": "^1.0.0",
 "@nestjs/common": "^9.0.0",
 "redis": "^4.6.6",
 "cache-manager-redis-store": "^3.0.1",

jlopez994 · 2023-06-03T18:04:46Z

This is my solution, works with latest version:

import { RedisClientOptions } from 'redis';
import { redisStore } from 'cache-manager-redis-yet';
import { ConfigModule, ConfigService } from '@nestjs/config';

CacheModule.registerAsync<RedisClientOptions>({
  imports: [ConfigModule],
  useFactory: async (config: ConfigService) => ({
    store: await redisStore({ url: config.get<string>('REDIS_URL') })
  }),
  inject: [ConfigService]
}),

"@nestjs/common": "^9.0.11",
"cache-manager": "^4.1.0",
"cache-manager-redis-yet": "^4.1.1",
"redis": "^4.6.7",

SirPhemmiey · 2023-07-01T22:24:37Z

It's also important to note that if you'll be importing this cache module from within app.module.ts, you have to set isGlobal to true

mengtongun · 2023-07-14T01:49:06Z

This is my solution, works with latest version implement same as @jlopez994 :

import { RedisClientOptions } from 'redis';
import { redisStore } from 'cache-manager-redis-yet';
import { ConfigModule, ConfigService } from '@nestjs/config';

CacheModule.registerAsync<RedisClientOptions>({
 imports: [ConfigModule],
 useFactory: async (cfg: ConfigService) => ({
   store: await redisStore({
    socket: {
      host: cfg.get('REDIS_HOST'),
      port: parseInt(cfg.get('REDIS_PORT ') || '6379'),
    },
    password: cfg.get('REDIS_PASSWORD'),
    ttl: cfg.get('REDIS_TTL'),
  }),
 }),
 inject: [ConfigService]
}),

"@nestjs/common": "^10.0.5",
"cache-manager": "^5.2.3",
"cache-manager-redis-yet": "^4.1.2",
"redis": "^4.6.7",

This worked for me.

CacheTTL() worked fine.
Cache Manager
set: (key: string, value: unknown, ttl?: number) also worked fine.

elouizbadr · 2023-07-15T11:16:39Z

This is my solution, works with latest version implement same as @jlopez994 :

import { RedisClientOptions } from 'redis';
import { redisStore } from 'cache-manager-redis-yet';
import { ConfigModule, ConfigService } from '@nestjs/config';

CacheModule.registerAsync<RedisClientOptions>({
 imports: [ConfigModule],
 useFactory: async (cfg: ConfigService) => ({
   store: await redisStore({
    socket: {
      host: cfg.get('REDIS_HOST'),
      port: parseInt(cgf.get('REDIS_PORT ') || '6379'),
    },
    password: cfg.get('REDIS_PASSWORD'),
    ttl: cfg.get('REDIS_TTL'),
  }),
 }),
 inject: [ConfigService]
}),

"@nestjs/common": "^10.0.5",
"cache-manager": "^5.2.3",
"cache-manager-redis-yet": "^4.1.2",
"redis": "^4.6.7",

This worked for me.

CacheTTL() worked fine.
Cache Manager
set: (key: string, value: unknown, ttl?: number) also worked fine.

Thank you @mengtongun for this solution. Just a side note, you've got a typo in the following line (Where cgf should be cfg):

port: parseInt(cgf.get('REDIS_PORT ') || '6379'),

SlinkyPotato · 2023-08-05T14:20:40Z

This is interesting.

A few things:

The last commit from this project was October 16 2022.

This project is forked to https://github.com/node-cache-manager/node-cache-manager-redis-yet#license

It is officially maintained by node-cache-manager according to NestJS docs https://docs.nestjs.com/techniques/caching#different-stores
https://github.com/node-cache-manager/node-cache-manager#store-engines

Is this project abandoned?

Here is a working example from node-cache-manager-redis-yet

import { CacheModule } from '@nestjs/cache-manager';
import { RedisClientOptions } from 'redis';
import { redisStore } from 'cache-manager-redis-yet';

@Module({
  imports: [
    CacheModule.register<RedisClientOptions>({
      store: redisStore,
      socket: {
        host: process.env.REDIS_HOST ?? 'localhost',
        port: parseInt(process.env.REDIS_PORT ?? '6379'),
      },
    }),
  ],
})
export class TestModule {}

...
constructor(@Inject(CACHE_MANAGER) private cacheManager: Cache) {}
...
await this.cacheManager.set('test', 'test-value');
const testVal = await this.cacheManager.get('test');
...

"@nestjs/common": "^10.0.5",
"cache-manager": "^5.2.3",
"cache-manager-redis-yet": "^4.1.2",
"redis": "^4.6.7",

Biigode · 2023-08-07T20:43:40Z

This is interesting.

A few things:

The last commit from this project was October 16 2022.

This project is forked to https://github.com/node-cache-manager/node-cache-manager-redis-yet#license

It is officially maintained by node-cache-manager according to NestJS docs https://docs.nestjs.com/techniques/caching#different-stores

Is this project abandoned?

Here is a working example from node-cache-manager-redis-yet

import { CacheModule } from '@nestjs/cache-manager';
import { RedisClientOptions } from 'redis';
import { redisStore } from 'cache-manager-redis-yet';

@Module({
  imports: [
    CacheModule.register<RedisClientOptions>({
      store: redisStore,
      socket: {
        host: process.env.REDIS_HOST ?? 'localhost',
        port: parseInt(process.env.REDIS_PORT ?? '6379'),
      },
    }),
  ],
})
export class TestModule {}

...
constructor(@Inject(CACHE_MANAGER) private cacheManager: Cache) {}
...
await this.cacheManager.set('test', 'test-value');
const testVal = await this.cacheManager.get('test');
...

"@nestjs/common": "^10.0.5",
"cache-manager": "^5.2.3",
"cache-manager-redis-yet": "^4.1.2",
"redis": "^4.6.7",

This solutions works for me thanks.
Follow the images

zenstok · 2023-08-22T13:36:16Z

Please be cautious regarding this issue that I came across while using cache-manager-redis-yet. This issue could potentially lead to a security vulnerability within your application, especially if you are depending on the @Exclude() class validator decorator to conceal attributes from the frontend.

NinjaMax · 2023-10-24T14:32:43Z

Maybe I can help someone.
In my case :

CacheTTL() doesn't worked.
Cache Manager
set: (key: string, value: unknown, ttl?: number) also doesn't worked.
return { store: redisStore as unknown as CacheStore, ttl: 5000}; doesn't worked.

With TTL - this is workes for me.

import { CacheStore, Module } from '@nestjs/common';
import { RedisClientOptions, createClient } from 'redis';
import { CacheModule } from '@nestjs/cache-manager';
//--------------------------------------------------------
CacheModule.registerAsync<RedisClientOptions>({
      imports: [ConfigModule.register({ folder: './config' })],
      useFactory: async (configService: ConfigService) => {
        const store = createClient({
          socket: {
            host: configService.get('REDIS_HOST'),
            port: +configService.get('REDIS_PORT'), 
          },
        });
        await store.connect().catch(console.error);
        const redisStore = new RedisStore({
          client: store,
          prefix: 'tyre-no-offset: ', 
          ttl: 5000,
        });
        return { store: redisStore as unknown as CacheStore };
      },
      isGlobal: true,
      inject: [ConfigService],
    }),
//--------------------------------------------

controller:

 @UseInterceptors(CacheInterceptor)
 @CacheKey('no-offset')
 @Get('/no-offset')
 findTyresWithoutLimit(@Query('width') width: string,) {
   return this.tyresService.findAllTyresWithoutOffset(width);
 }

mohit4bug · 2024-04-15T18:23:22Z

Seems to work!

import { AppController } from "@/app.controller"
import { AppService } from "@/app.service"
import { CacheModule } from "@nestjs/cache-manager"
import { Module } from "@nestjs/common"
import { ConfigModule, ConfigService } from "@nestjs/config"
import { redisStore } from "cache-manager-redis-yet"
import type { RedisClientOptions } from "redis"

@Module({
  imports: [
    CacheModule.registerAsync<RedisClientOptions>({
      imports: [ConfigModule],
      inject: [ConfigService],
      useFactory: async (configService: ConfigService) => ({
        store: await redisStore({
          ttl: parseInt(configService.get<string>("REDIS_TTL")),
          socket: {
            host: configService.get<string>("REDIS_HOST"),
            port: parseInt(configService.get<string>("REDIS_PORT")),
          },
        }),
      }),
      isGlobal: true,
    }),
    ConfigModule.forRoot({
      isGlobal: true,
    }),
  ],
  controllers: [AppController],
  providers: [AppService],
})
export class AppModule {}

ygweric · 2024-04-29T06:39:39Z

set: (key: string, value: unknown, ttl?: number) also doesn't worked.

same bug with me

micalevisk mentioned this issue May 26, 2022

Caching with CacheManager and Redis: custom host and port are ignored in register() nestjs/docs.nestjs.com#2324

Closed

5 tasks

FoxxMD mentioned this issue Sep 7, 2022

Upgrade to use redis v4 #43

Closed

2 tasks

dabroek mentioned this issue Oct 17, 2022

Update caching example conforming to the new cache-manager-redis-store interface nestjs/docs.nestjs.com#2498

Open

10 tasks

biud436 mentioned this issue Dec 22, 2022

TODO: handle the rss using redis cache between blue and green servers. biud436/blog-api-server#9

Closed

This comment was marked as resolved.

Sign in to view

yusuftaufiq mentioned this issue Feb 14, 2023

Incompatible with NestJS cache store #53

Open

develjsw mentioned this issue Aug 13, 2023

redis ttl 설정 이슈 develjsw/nestjs-api#9

Open

tokdaniel mentioned this issue Nov 3, 2023

Feature: Cache with Redis superfluid-finance/stripe-integration#20

Merged

lutzzdias mentioned this issue Nov 6, 2023

Implement auth lutzzdias/artisync#37

Closed

lsh23 mentioned this issue Dec 8, 2023

redis option 수정 boostcampwm2023/iOS02-moti#534

Merged

Upgrade redis to v4 #40

Upgrade redis to v4 #40

Comments

dantehemerson commented Apr 20, 2022

sarcastic-verma commented Jun 28, 2022

vxm5091 commented Jun 30, 2022

Sravanchittimalla commented Jul 15, 2022

celianvdb commented Jul 18, 2022

dabroek commented Jul 18, 2022

FoxxMD commented Sep 1, 2022

muratdemirci commented Sep 5, 2022

tvacherat commented Sep 26, 2022

andreaziz commented Oct 11, 2022

smentek commented Oct 12, 2022

alibeknow commented Oct 13, 2022

mehamednews commented Oct 14, 2022

dabroek commented Oct 15, 2022

DmytroHaponovMD commented Oct 17, 2022

dabroek commented Oct 17, 2022

NarHakobyan commented Oct 20, 2022

smentek commented Oct 20, 2022

NarHakobyan commented Oct 20, 2022

FoxxMD commented Oct 20, 2022

smentek commented Oct 20, 2022

assisgui commented Oct 20, 2022

cskiwi commented Oct 21, 2022 • edited

SerhiiBabich commented Oct 22, 2022

mihailgolban commented Oct 27, 2022

SerhiiBabich commented Oct 27, 2022

NarHakobyan commented Oct 27, 2022 • edited

conioX commented Nov 3, 2022

kamal-brill commented Nov 23, 2022

codeprefect commented Nov 27, 2022

kamal-brill commented Nov 28, 2022

codeprefect commented Nov 29, 2022

codeprefect commented Nov 29, 2022

This comment was marked as resolved.

zbingsong commented Jan 14, 2023

wodCZ commented Jan 15, 2023

How to use redis with NestJS cache manager

Notes about TTL

puzzle9 commented Jan 26, 2023 • edited

How to use redis with NestJS cache manager

Notes about TTL

ajfranzoia commented Mar 20, 2023

anilahir commented Apr 3, 2023

Justin3go commented May 14, 2023 • edited

Akupsmee commented Jun 2, 2023 • edited

igmrrf commented Jun 3, 2023 • edited

jlopez994 commented Jun 3, 2023

SirPhemmiey commented Jul 1, 2023 • edited

mengtongun commented Jul 14, 2023 • edited

elouizbadr commented Jul 15, 2023

SlinkyPotato commented Aug 5, 2023 • edited

Biigode commented Aug 7, 2023 • edited

zenstok commented Aug 22, 2023

NinjaMax commented Oct 24, 2023

mohit4bug commented Apr 15, 2024

Seems to work!

ygweric commented Apr 29, 2024

cskiwi commented Oct 21, 2022 •

edited

NarHakobyan commented Oct 27, 2022 •

edited

puzzle9 commented Jan 26, 2023 •

edited

Justin3go commented May 14, 2023 •

edited

Akupsmee commented Jun 2, 2023 •

edited

igmrrf commented Jun 3, 2023 •

edited

SirPhemmiey commented Jul 1, 2023 •

edited

mengtongun commented Jul 14, 2023 •

edited

SlinkyPotato commented Aug 5, 2023 •

edited

Biigode commented Aug 7, 2023 •

edited