Cache expire patterns

[sidinwillis]

Hi, thank you for sharing!

I think there's a paranthesis here put in the wrong place:

const patterns = keysPattern.split(',').map(
     pattern => pattern.endsWith('*') ? pattern : [pattern, pattern + DATA_POSTFIX]) //<--here
    .reduce((acc, item) =>
    {           
        acc.push(...item)
        return acc          
    }, [])

If the pattern is

*/test*

the reduce would make this an array of char.

The first character (*) is then passed to getkeys, deleting the entire database.

I fixed it in this way

const patterns = keysPattern.split(',').map(
     pattern => pattern.endsWith('*') ? pattern : [pattern, pattern + DATA_POSTFIX]
    .reduce((acc, item) =>
    {           
        acc.push(...item)
        return acc          
    }, [])) <--here

I wrong something?

[jkyberneees]

Hi @sidinwillis thanks for reporting.
I have pushed the fix up, could you please review and comment before merging?

Regards

[sidinwillis]

Thanks to you!

Great, actually patterns is an array so your solution works fine.

I've tested in local env and work well.