[MB-30176] Quoted strings with escaped characters inside parenthesis raise syntax error wrongly - Couchbase database

XML

Word

Printable

Details

Type: Bug
Status: Closed
Priority: Major
Resolution: Fixed
Affects Version/s: 4.0.0, 4.1.0, 4.5.0, 4.6.0, 5.0.0, 5.1.0, 6.5.0, 6.0.0, 5.5.0
Fix Version/s: 7.0.0
Component/s: query
Labels:
- customer

Triage:
Untriaged
Is this a Regression?:
Unknown

Description

Queries like

select x from default where contains(x, "
"")

fail with a syntax error (end of input)

('\'')

fails too with invalid quoted string.

Attachments

Issue Links

backports to

MB-46262 Backport MB-30176 to 6.6.3

Closed

Gerrit Reviews

- Issue Only
- Show All Reviews
- Show Open Reviews
- Show All Issues
- Show Open Issues

No reviews matched the request. Check your Options in the drop-down menu of this sections header.

Activity

Ascending order - Click to sort in descending order

View 7 older comments

Donald Haggart added a comment - 17/May/21 10:35 AM

Pierre Regazzoni, I'm not sure I follow... in the last example you have an opening single quote plus escaped literal single quote and no closing single quote, hence the syntax error. You can use "'" (enclose the single quote in double quotes) or '''' (enclose an escaped-using-character-doubling single quote in single quotes).

Remember there is a special case for single quotes where {{'' }}=={{ \'}} when inside a string.

cbq> select '''';

...

"results": [

 "$1": "'"

...

Donald Haggart added a comment - 17/May/21 10:35 AM Pierre Regazzoni , I'm not sure I follow... in the last example you have an opening single quote plus escaped literal single quote and no closing single quote, hence the syntax error. You can use "'" (enclose the single quote in double quotes) or '''' (enclose an escaped-using-character-doubling single quote in single quotes). Remember there is a special case for single quotes where {{'' }} == {{ \'}} when inside a string. cbq> select ''''; ... "results": [ { "$1": "'" ...

Donald Haggart added a comment - 17/May/21 10:37 AM

(FWIW, this escaping-by-doubling isn't my favourite but it was already in the code so... legacy...)

Donald Haggart added a comment - 17/May/21 10:37 AM (FWIW, this escaping-by-doubling isn't my favourite but it was already in the code so... legacy...)

Pierre Regazzoni added a comment - 17/May/21 10:40 AM

ok i guess in the case of ''' syntax error is expected since we interpret as ' + escaped(') and we don't see the closing '

Pierre Regazzoni added a comment - 17/May/21 10:40 AM ok i guess in the case of ''' syntax error is expected since we interpret as ' + escaped(') and we don't see the closing '

Pierre Regazzoni added a comment - 17/May/21 11:25 AM

closing per discussion

Pierre Regazzoni added a comment - 17/May/21 11:25 AM closing per discussion

Couchbase Build Team added a comment - 27/Jan/22 9:14 AM

Build couchbase-server-7.1.0-2173 contains query commit fbb422f with commit message:
~~MB-50619~~ Revise ~~MB-30176~~ fix.

Couchbase Build Team added a comment - 27/Jan/22 9:14 AM Build couchbase-server-7.1.0-2173 contains query commit fbb422f with commit message: MB-50619 Revise MB-30176 fix.