Repository navigation
Calculator SELECT via query endpoint problem #580
Description
ArcadeDB Version: 22.9.1 (build d082adc/1664822987697/main)
JDK Version: openjdk version "11.0.16" 2022-07-19
OS: Ubuntu 20.04
Using SELECT 1 + 1 via endpoint gives an error, while via console and studio this works. In the studio it seems to work, as the "command" endpoint is used.
Expected behavior
2
Actual behavior
{"exception":"com.arcadedb.query.sql.parser.ParseException","detail":"Encountered \" <INTEGER_LITERAL> \"1 \"\" at line 1, column 12. Was expecting one of: <EOF> <LIMIT> ... <SKIP2> ... <OFFSET> ... <TIMEOUT> ... <AS> ... <LET> ... <UNWIND> ... \";\" ... \",\" ... \":\" ... \":\" ... \":\" ... \":\" ... \":\" ... \":\" ... \":\" ... \":\" ... \":\" ... \":\" ... \":\" ... \":\" ... <AS> ... \",\" ... \":\" ... \":\" ... \":\" ... \":\" ... \":\" ... \":\" ... \":\" ... \":\" ... \":\" ... \":\" ... \":\" ... <AS> ... <AS> ... <AS> ... <AS> ... <AS> ... <AS> ... <AS> ... \":\" ... \":\" ... \":\" ... \":\" ... \":\" ... \":\" ... \":\" ... \":\" ... \":\" ... \":\" ... ","error":"Cannot execute command"}
Steps to reproduce
curl -X GET http://localhost:2480/api/v1/query/school/sql/select%201%20%2B%201 --user root:arcadedb-password
Activity
gramian commentedon Nov 3, 2022
Likely related to #566 .gramian commentedon Nov 21, 2022
@lvca
This error depends on the endpoint: the
commandendpoint can doSELECT 1 + 1, while thequeryendpoint throws the error above. Maybe this becomes fixable given this observation?lvca commentedon Nov 28, 2022
It could be dependent on the encoding. While the command gets the query in POST, the query command gets the query in the URL and it must be encoded. Trying to create a test case to reproduce it.
[-]Calculator SELECT via endpoint problem[/-][+]Calculator SELECT via query endpoint problem[/+]gramian commentedon Dec 12, 2022
This problem seems to be specific to
+. I think this could be related to this comment. However, when usinginstead of
URLDecoder.decode(text, exchange.getRequestCharset())anunreported exception java.net.URISyntaxExceptionerror is reported during compilation. I am not familiar enough with Java or this code base to add a useful exception catch.Fixed issue with HTTP het /query API and encoding of the command
lvca commentedon Dec 12, 2022
The problem with that piece of code is that text is not a URI, but only the command text. Found and fixed the issue. It was the decoding. The text arrives already decoded, so that line not only is not necessary, but it removed the
+character.chore(studio-deps-dev)(deps-dev): bump enhanced-resolve from 5.21.2 t…
Fixed issue with HTTP het /query API and encoding of the command
chore(studio-deps-dev)(deps-dev): bump enhanced-resolve from 5.21.2 t…
chore(deps): bump the github-actions group across 1 directory with 8 …