express-brute-redis

express-brute-redis copied to clipboard

Not working with >[email protected] redis

1

Express-Brute: 1.0.1 Node: 16.13.1 Redis: 4.3.0 With new Redis, there is no support for the callback anymore, please see the [redis npm](https://www.npmjs.com/package/redis)

Code-Crash

Listen for redis client results and throw error on failure

3

When using `express-brute` backed by `express-brute-redis` as the store I ran in to problems when my redis server would become inaccessible (for some temporary network glitch or something similar). I...

peralmq

You are updating the value in the Redis key with `set` whereby the value is passed from the calling `express-brute` module. But the value might have changed in the meantime....

Fritte

I'm using ioredis, and I"d prefer not to have use nod-redis as ioredis has more extensive support for e.g. clustering. Perhaps redis should be made a peer module so that...

pszabop

holm

Note there was an existing flakey test. Fixes https://github.com/AdamPflug/express-brute-redis/issues/3

facultymatt

When creating a new redis client the thrid argument is this.options.redisOptions which incorrect is undefined. It should be `this.redisOptions` which is the object created for this purpose. This makes it...

facultymatt