ts.mRangeWithLabels FILTER as an array of string not apply

[PierreMasson]

ts.mRangeWithLabels no longer takes the FILTER parameter into account as an array.

ERR TSDB: missing labels for filter argument

Downgrading to redis 4.1.0 in my package.json do the job.

Maybe this commit #5dd7d3149a633b762ece0ce8006ecbb4311a547f is to blame?

Environment:

• Node.js Version: 14.18.1
• Redis Server Version: 6.2.7
• Node Redis Version: 4.3.1
• Platform: Debian GNU/Linux 11 (bullseye)

[leibale]

It's the same bug as #2251 (you can read more about what happened there). I'll release a new version soon with a fix for that as well.

[Shepless]

@leibale any ETA on the fix being released please?

[leibale]

@Shepless it's not in redis@4.4.0?

[Shepless]

@Shepless it's not in redis@4.4.0?

@leibale I still get a similar error on redis@4.4.0 when using the time series package @redis/time-series@1.0.3

ERR TSDB: missing labels for filter argument

Code snippet:

const labels = [`exchange=${exchange}`, `base=${symbol.base}`, `quote=${symbol.quote}`, `interval=${interval}`]
const data = await this.client.ts.mRangeWithLabels(from, to, labels);

If I downgrade to redis@4.1.0 it works fine. Sorry if this is a separate issue, I just thought they would have been caused by the same problem?

EDIT
If I use redis@4.4.0 and I only pass a single label as a string it doesn't error (but I need to filter with multiple labels)

[Shepless]

@leibale sorry to keep bothering you, is the above related to the same issue? If not, do you want me to open a new issue for it? Thank you, all of your work and effort is appreciated 😃

[leibale]

@Shepless sorry about that, should be fixed in redis@4.5.0/@redis/time-series@1.0.4 :)

[TombolaShepless]

Thanks @leibale will give it a try this evening!