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.

Sign up for GitHub

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

Jump to bottom

Empty PARAMS causes RediSearch fail #2646

Open
Didas-git opened this issue Nov 4, 2023 · 0 comments · May be fixed by #2647
Open

Empty PARAMS causes RediSearch fail #2646

Didas-git opened this issue Nov 4, 2023 · 0 comments · May be fixed by #2647
Labels
Bug

Comments

@Didas-git
Copy link
Contributor

Didas-git commented Nov 4, 2023 •
edited

Description

When calling ft.search and passing PARAMS: {} to the node-redis options it adds PARAMS 0 to the query which its invalid, it should not add anything to the query

Node.js Version

20.5

Redis Server Version

7.2

Node Redis Version

4.6.10

Platform

Windows & Linux

Logs

`[ErrorReply: Parameters must be specified in PARAM VALUE pairs]`
@Didas-git Didas-git added the Bug label Nov 4, 2023
Didas-git added a commit to Didas-git/node-redis that referenced this issue Nov 4, 2023
@Didas-git
Fix redis#2646 for v4
d90e06c 
Auto formatter did some thingy...
@Didas-git Didas-git linked a pull request Nov 4, 2023 that will close this issue
Open
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix #2646 for v4 Didas-git/node-redis
1 participant
@Didas-git