diff options
author | Martin Polden <mpolden@mpolden.no> | 2022-08-26 13:45:38 +0200 |
---|---|---|
committer | Martin Polden <mpolden@mpolden.no> | 2022-08-29 10:21:57 +0200 |
commit | 7587a999ce58215b344fe647a361be0d77c9bc39 (patch) | |
tree | 488634fd098a337dac4ac8cb35806e0f8b472a85 /controller-server/src/main/java/com/yahoo/vespa/hosted/controller/restapi/configserver/ConfigServerApiHandler.java | |
parent | 12817c4924a6809bd6b992591e655e5ab154308e (diff) |
Do not include exception message when an unexpected error happens
Diffstat (limited to 'controller-server/src/main/java/com/yahoo/vespa/hosted/controller/restapi/configserver/ConfigServerApiHandler.java')
-rw-r--r-- | controller-server/src/main/java/com/yahoo/vespa/hosted/controller/restapi/configserver/ConfigServerApiHandler.java | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/restapi/configserver/ConfigServerApiHandler.java b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/restapi/configserver/ConfigServerApiHandler.java index 467b0c094cc..e764fed4653 100644 --- a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/restapi/configserver/ConfigServerApiHandler.java +++ b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/restapi/configserver/ConfigServerApiHandler.java @@ -17,11 +17,11 @@ import com.yahoo.vespa.hosted.controller.api.integration.zone.ZoneRegistry; import com.yahoo.vespa.hosted.controller.auditlog.AuditLoggingRequestHandler; import com.yahoo.vespa.hosted.controller.proxy.ConfigServerRestExecutor; import com.yahoo.vespa.hosted.controller.proxy.ProxyRequest; +import com.yahoo.vespa.hosted.controller.restapi.ErrorResponses; import com.yahoo.yolean.Exceptions; import java.net.URI; import java.util.List; -import java.util.logging.Level; import java.util.stream.Collectors; import java.util.stream.Stream; @@ -69,9 +69,7 @@ public class ConfigServerApiHandler extends AuditLoggingRequestHandler { } catch (IllegalArgumentException e) { return ErrorResponse.badRequest(Exceptions.toMessageString(e)); } catch (RuntimeException e) { - log.log(Level.WARNING, "Unexpected error handling '" + request.getUri() + "', " - + Exceptions.toMessageString(e)); - return ErrorResponse.internalServerError(Exceptions.toMessageString(e)); + return ErrorResponses.logThrowing(request, log, e); } } |