Fix FalkorDB sanitize to handle forward slash character #1118
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
/to the separator map in FalkorDB'ssanitize()methodProblem
When entity names contained forward slashes (e.g., "Cruise / En route"), RediSearch queries were failing with syntax errors because the forward slash was not being sanitized. This resulted in malformed queries like:
Solution
Added
/to theseparator_mapin thesanitize()method (line 324 infalkordb_driver.py), treating it like other special characters by replacing it with whitespace.Now "Cruise / En route" is sanitized to "Cruise En route", which produces valid RediSearch queries.
Testing
Verified that:
Fixes data insertion issues when using FalkorDB backend with forward slash characters in entity names.