fix: correctly escape backslash in queries (#10522)

# Which Problems Are Solved

While investigating a support ticket, it was discovered that some
queries using equals or not equals without case matching were not
correctly escaping the value to compare. If a value contained a
backslash (`\`) the row would not match.

# How the Problems Are Solved

- Fixed the escaping for backslash for `like` operations.
- Changed equals and not equals comparison without case matching to `=`
instead of `like`.

# Additional Changes

None

# Additional Context

- related to a support request
- requires backport to v.3 and v4.x

(cherry picked from commit 6c8d027e72)
This commit is contained in:
Livio Spring
2025-08-21 07:25:23 +02:00
parent e9bf92e987
commit 95848219d5
3 changed files with 26 additions and 14 deletions

View File

@@ -862,7 +862,7 @@ func TestNewTextQuery(t *testing.T) {
},
want: &textQuery{
Column: testCol,
Text: "hu\\%rst",
Text: "hu%rst",
Compare: TextEqualsIgnoreCase,
},
},
@@ -875,7 +875,7 @@ func TestNewTextQuery(t *testing.T) {
},
want: &textQuery{
Column: testCol,
Text: "hu\\_rst",
Text: "hu_rst",
Compare: TextEqualsIgnoreCase,
},
},
@@ -888,7 +888,7 @@ func TestNewTextQuery(t *testing.T) {
},
want: &textQuery{
Column: testCol,
Text: "h\\_urst\\%",
Text: "h_urst%",
Compare: TextEqualsIgnoreCase,
},
},
@@ -914,7 +914,7 @@ func TestNewTextQuery(t *testing.T) {
},
want: &textQuery{
Column: testCol,
Text: "h\\_urst\\%",
Text: "h_urst%",
Compare: TextNotEqualsIgnoreCase,
},
},
@@ -1204,7 +1204,7 @@ func TestTextQuery_comp(t *testing.T) {
Compare: TextEqualsIgnoreCase,
},
want: want{
query: sq.Like{"LOWER(test_table.test_col)": "hurst"},
query: sq.Eq{"LOWER(test_table.test_col)": "hurst"},
},
},
{
@@ -1226,7 +1226,7 @@ func TestTextQuery_comp(t *testing.T) {
Compare: TextNotEqualsIgnoreCase,
},
want: want{
query: sq.NotLike{"LOWER(test_table.test_col)": "hurst"},
query: sq.NotEq{"LOWER(test_table.test_col)": "hurst"},
},
},
{
@@ -1237,7 +1237,18 @@ func TestTextQuery_comp(t *testing.T) {
Compare: TextEqualsIgnoreCase,
},
want: want{
query: sq.Like{"LOWER(test_table.test_col)": "hu\\%\\%rst"},
query: sq.Eq{"LOWER(test_table.test_col)": "hu%%rst"},
},
},
{
name: "equals ignore case backslash",
fields: fields{
Column: testCol,
Text: "AD\\Hurst",
Compare: TextEqualsIgnoreCase,
},
want: want{
query: sq.Eq{"LOWER(test_table.test_col)": "ad\\hurst"},
},
},
{
@@ -1255,11 +1266,11 @@ func TestTextQuery_comp(t *testing.T) {
name: "starts with wildcards",
fields: fields{
Column: testCol,
Text: "_Hurst%",
Text: "_Hur\\st%",
Compare: TextStartsWith,
},
want: want{
query: sq.Like{"test_table.test_col": "\\_Hurst\\%%"},
query: sq.Like{"test_table.test_col": "\\_Hur\\\\st\\%%"},
},
},
{