From 46fec84b5537e3153e82f4cc78a251481ed6d8a8 Mon Sep 17 00:00:00 2001 From: "francesco.mangiacrapa" Date: Fri, 27 Sep 2019 15:17:06 +0200 Subject: [PATCH] Only improved some messages --- .../datatransfer/resolver/services/StorageHubResolver.java | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/main/java/org/gcube/datatransfer/resolver/services/StorageHubResolver.java b/src/main/java/org/gcube/datatransfer/resolver/services/StorageHubResolver.java index 3abfbce..f299436 100644 --- a/src/main/java/org/gcube/datatransfer/resolver/services/StorageHubResolver.java +++ b/src/main/java/org/gcube/datatransfer/resolver/services/StorageHubResolver.java @@ -123,9 +123,9 @@ public class StorageHubResolver { dispValue = ConstantsResolver.CONTENT_DISPOSITION_VALUE.valueOf(contDisp); }catch (Exception e) { String allowedValues = String.format("{%s,%s}", CONTENT_DISPOSITION_VALUE.inline,CONTENT_DISPOSITION_VALUE.attachment); - String errorMsg = "The parameter "+ConstantsResolver.QUERY_PARAM_CONTENTDISPOSITION+" passed in the query string is not a value of "+allowedValues; - logger.error(errorMsg); - throw ExceptionManager.badRequestException(req, errorMsg, StorageHubResolver.class, help); + String msg = String.format("Wrong parameter: '%s=%s'. Valid '%s' values are: %s",ConstantsResolver.QUERY_PARAM_CONTENTDISPOSITION,contDisp,ConstantsResolver.QUERY_PARAM_CONTENTDISPOSITION,allowedValues); + logger.error(msg); + throw ExceptionManager.wrongParameterException(req, msg, StorageHubResolver.class, help); } }