Parenthesis in a search phrase will throw an error

neo4j-devtools / neo4j-bloom

A public repository for informal docs, problem reporting and content sharing related to Neo4j Bloom.

Apache License 2.0

18 stars 0 forks source link

Parenthesis in a search phrase will throw an error #22

Closed M3nt3 closed 3 years ago

M3nt3 commented 3 years ago

apparently adding parenthesis to a serach frase will make the query fail. The only difference between the snapshots is that I removed the parenthesis and added a coma in the "Search Phrase" field.

jharris4 commented 3 years ago

@M3nt3 thanks for reporting the issue! We're not seeing this issue against newer versions of Neo4j. Would you mind sharing with us the version of the database you were running Bloom against when you encountered this issue?

M3nt3 commented 3 years ago

Hello and thanks for answering. We are running Bloom Server (Neo4j Bloom version: 1.3.1) and I am connected through Chrome.

On top of the above, I noticed a related but different behaviour, that cause a worse crash. I created a new Search with parenthesis, but this time I put the parameter within parenthesis:

Now I cannot run any search at all, as the search bar times out without matching any search phrase. This as soon as I start typing anything at all, even a normal query and not a search phrase. Bloom remains freezed until I actually modify that search phrase and remove the parenthesis.

If I move the parameter outside those parentheses, I reproduce the same as above and only this query fails, with no timeout:

Thanks for checking this out.

jharris4 commented 3 years ago

Hi, thanks for the clear instructions! We can reproduce the new error you mentioned and we will investigate further.

As for the original issue, sorry I should have been more clear! Can you please report the version of the database (and not Bloom version) when you see the original error?

You can find the Neo4j database version from the Perspective Gallery like in the screenshot below:

This can be opened by clicking the top button in the menu bar on the left, and then clicking the title/name of your current perspective at the top of the panel that is open...

M3nt3 commented 3 years ago

Hi! Apologise, did not think about specifying the Neo4j version. We are running 4.0.4

Thanks for looking into it. It is not a huge problem avoiding parenthesis to be on the safe side, but I can see this being a nasty bug to find out as it is strange to think it comes from spelling (probably related to the parenthesis notation of nodes in cypher?).

jharris4 commented 3 years ago

Yeah, parenthesis notation can be tricky both because of its use in cypher, and because it is used for grouping in regular expressions.

Thanks for providing your Neo4j version, it'll help us narrow down the reproduction for the original/first issue you reported.

jharris4 commented 3 years ago

@M3nt3 We have just released Bloom version 1.5.0 which includes some fixes for special characters in search phrase text. If you have a moment to try it out it would be great if you could let us know if you can still reproduce the issue. Thanks!

joknelid commented 3 years ago

Tested with 1.6.1 and works fine, going to close this issue