query_es_dsl: force query to be fully parsed

or fail.

nrt added

Change-Id: Ibe9c3252780eff3281df3829c6a2509b421cca09

dci/analytics/query_es_dsl.py

@@ -81,7 +81,7 @@
 
 
 def parse(q):
-    return query.parseString(q).asList()
+    return query.parseString(q, parseAll=True).asList()
 
 
 _op_to_es_op = {"<": "lt", "<=": "lte", ">": "gt", ">=": "gte"}

dci/api/v1/analytics.py

@@ -321,7 +321,7 @@ def tasks_jobs(user):
         es_query = build_es_query(args)
     except Exception as e:
         logger.error(f"error while building the elastic query: {e}")
-        raise dci_exc.DCIException("error while building the elastic query")
+        raise dci_exc.DCIException("syntax error while parsing the query")
 
     try:
         res = requests.get(

tests/analytics/test_query_es_dsl.py

@@ -24,6 +24,9 @@ def test_parse_query_invalid():
     with pytest.raises(pp.ParseException):
         qed.parse("toto")
 
+    with pytest.raises(pp.ParseException):
+        qed.parse("(toto=titi) and (lol=mdr")
+
 
 def test_parse_query_valid():
     ret = qed.parse("f1=v1")