-
Notifications
You must be signed in to change notification settings - Fork 2.6k
Discrepancy in INFO
response parsing
#3262
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.
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
Comments
gerzse
added a commit
to gerzse/redis-py
that referenced
this issue
Jun 5, 2024
If the INFO response contains a single `a=b` value for any of the keys, that must also be parsed into a dictionary. Fixes redis#3262
gerzse
added a commit
that referenced
this issue
Jun 5, 2024
If the INFO response contains a single `a=b` value for any of the keys, that must also be parsed into a dictionary. Fixes #3262 Co-authored-by: Gabriel Erzse <[email protected]>
gerzse
added a commit
to gerzse/redis-py
that referenced
this issue
Jun 5, 2024
If the INFO response contains a single `a=b` value for any of the keys, that must also be parsed into a dictionary. Fixes redis#3262
gerzse
added a commit
that referenced
this issue
Jun 5, 2024
If the INFO response contains a single `a=b` value for any of the keys, that must also be parsed into a dictionary. Fixes #3262 Co-authored-by: Gabriel Erzse <[email protected]>
agnesnatasya
pushed a commit
to agnesnatasya/redis-py
that referenced
this issue
Jul 20, 2024
If the INFO response contains a single `a=b` value for any of the keys, that must also be parsed into a dictionary. Fixes redis#3262 Co-authored-by: Gabriel Erzse <[email protected]>
vladvildanov
pushed a commit
that referenced
this issue
Sep 27, 2024
If the INFO response contains a single `a=b` value for any of the keys, that must also be parsed into a dictionary. Fixes #3262 Co-authored-by: Gabriel Erzse <[email protected]>
vladvildanov
pushed a commit
that referenced
this issue
Sep 27, 2024
If the INFO response contains a single `a=b` value for any of the keys, that must also be parsed into a dictionary. Fixes #3262 Co-authored-by: Gabriel Erzse <[email protected]>
vladvildanov
pushed a commit
that referenced
this issue
Sep 27, 2024
If the INFO response contains a single `a=b` value for any of the keys, that must also be parsed into a dictionary. Fixes #3262 Co-authored-by: Gabriel Erzse <[email protected]>
Kakadus
pushed a commit
to Kakadus/redis-py
that referenced
this issue
Jan 10, 2025
If the INFO response contains a single `a=b` value for any of the keys, that must also be parsed into a dictionary. Fixes redis#3262 Co-authored-by: Gabriel Erzse <[email protected]> Signed-off-by: Salvatore Mesoraca <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Version:
5.0.0
Description: The code for parsing the
INFO
response is creating a discrepancy in the new RediSearch index field types field, e.g.,search_fields_text
.If there is more than one section to this field its corresponding value (e.g.,
TEXT=1, SORTABLE=1
) is returned as a dictionary (as expected), while if there is only one value for it (e.g.,TEXT=1
) it is returned as a string!This is bad since we later use this value assuming that it is a dictionary, and thus face failures in the case that it is not.
The wanted behavior: We always get a dictionary back, whether we have only one section for this value or more.
Example: We have one section for the

TEXT
field, and two for theTAG
field, and this is what we get:The source of the problem is in the
parse_info()
function, here.Thanks!
The text was updated successfully, but these errors were encountered: