Feat: hide partition fitler in exception

gisaia · Feb 26, 2025 · be335bc · be335bc
1 parent c0fc153
commit be335bc
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/arlas-core/src/main/java/io/arlas/server/core/utils/ParamsParser.java b/arlas-core/src/main/java/io/arlas/server/core/utils/ParamsParser.java
@@ -49,7 +49,8 @@
 import java.util.regex.Pattern;
 import java.util.stream.Collectors;
 import java.util.stream.IntStream;
-
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 import static io.arlas.server.core.services.FluidSearchService.*;
 import static io.arlas.server.core.utils.CheckParams.GEO_AGGREGATION_TYPE_ENUMS;
 
@@ -80,6 +81,7 @@ public class ParamsParser {
     public static final String RANGE_ALIASES_CHARACTER = "$";
     public static final String TIMESTAMP_ALIAS = "timestamp";
     public static final String BAD_FIELD_ALIAS = "This alias does not represent a collection configured field. ";
+    private static Logger LOGGER = LoggerFactory.getLogger(ParamsParser.class);
 
 
     public static List<Aggregation> getAggregations(CollectionReference collectionReference, List<String> agg) throws ArlasException {
@@ -274,8 +276,9 @@ public static List<Filter> getPartitionFilter(CollectionReference collectionRefe
                 try {
                     fList = objectMapper.readValue(sf, new TypeReference<List<Filter>>() {});
                 } catch (JsonProcessingException ex) {
-
-                    throw new InvalidParameterException(INVALID_FILTER + ": '" + sf + "'", ex);
+                    // We dont print the filter in the exception to hide it from the final user
+                    LOGGER.error("{} : '{}'", INVALID_FILTER,sf);
+                    throw new InvalidParameterException(INVALID_FILTER,ex);
                 }
             }
             return fList;