From e193c80d110a07380e2e598ab098b80740324c80 Mon Sep 17 00:00:00 2001 From: lucio Date: Fri, 2 Feb 2024 10:54:19 +0100 Subject: [PATCH] removed logs --- src/main/java/org/gcube/smartgears/Bootstrap.java | 11 ++++++++--- .../application/request/RequestMetrics.java | 13 +++++++------ 2 files changed, 15 insertions(+), 9 deletions(-) diff --git a/src/main/java/org/gcube/smartgears/Bootstrap.java b/src/main/java/org/gcube/smartgears/Bootstrap.java index 9f107a6..599e1b8 100644 --- a/src/main/java/org/gcube/smartgears/Bootstrap.java +++ b/src/main/java/org/gcube/smartgears/Bootstrap.java @@ -89,10 +89,13 @@ public class Bootstrap implements ServletContainerInitializer { @SuppressWarnings("resource") private void initialiseContainer() { + ClassLoader contextCL = Thread.currentThread().getContextClassLoader(); + try { + // TODO Ask why is needed? + Thread.currentThread().setContextClassLoader(ContainerManager.class.getClassLoader()); log.trace("smartgears is starting"); - /* Get the ContainerContext. Look at DefaultProvider */ context = ProviderFactory.provider().containerContext(); @@ -108,7 +111,7 @@ public class Bootstrap implements ServletContainerInitializer { new ProcessorMetrics().bindTo(registry); Metrics.addRegistry(registry); - + /* Validate the configuration retrieved by ContainerContext * using gcube facilities annotation based @@ -124,6 +127,8 @@ public class Bootstrap implements ServletContainerInitializer { //we let the container continue + } finally {//restore the classloader of the current application + Thread.currentThread().setContextClassLoader(contextCL); } } @@ -141,7 +146,7 @@ public class Bootstrap implements ServletContainerInitializer { try { // TODO Ask why is needed? - //Thread.currentThread().setContextClassLoader(ContainerManager.class.getClassLoader()); + Thread.currentThread().setContextClassLoader(ContainerManager.class.getClassLoader()); manager = ContainerManager.instance; diff --git a/src/main/java/org/gcube/smartgears/handlers/application/request/RequestMetrics.java b/src/main/java/org/gcube/smartgears/handlers/application/request/RequestMetrics.java index bf07ad2..0cb9075 100644 --- a/src/main/java/org/gcube/smartgears/handlers/application/request/RequestMetrics.java +++ b/src/main/java/org/gcube/smartgears/handlers/application/request/RequestMetrics.java @@ -36,12 +36,13 @@ public class RequestMetrics extends RequestHandler { @Override public void handleResponse(ResponseEvent e) { - String statusCode = Integer.toString(e.response().getStatus()); - Metrics.timer(HTTP_REQUEST_METRICS_NAME, "status", statusCode).record(Duration.ofMillis(System.currentTimeMillis() - startCallThreadLocal.get())); - startCallThreadLocal.remove(); - log.debug("metrics for current call saved"); + try { + String statusCode = Integer.toString(e.response().getStatus()); + Metrics.timer(HTTP_REQUEST_METRICS_NAME, "status", statusCode).record(Duration.ofMillis(System.currentTimeMillis() - startCallThreadLocal.get())); + startCallThreadLocal.remove(); + }catch(Throwable t) { + log.warn("error getting Metrics",t); + } } - - }