Add Null check to second processRequest (#631)

Co-authored-by: Giorgio Trettenero <[email protected]>
Netflix · Feb 14, 2025 · 7841ff6 · 7841ff6
1 parent cb03b4a
commit 7841ff6
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/metacat-main/src/main/java/com/netflix/metacat/main/api/RequestWrapper.java b/metacat-main/src/main/java/com/netflix/metacat/main/api/RequestWrapper.java
@@ -154,7 +154,6 @@ public <R> R processRequest(
         if (requestTags != null) {
             tags.putAll(requestTags);
         }
-
         tags.put("request", resourceRequestName);
         tags.put("scheme", MetacatContextManager.getContext().getScheme());
         String clientAppName =  MetacatContextManager.getContext().getClientAppName();
@@ -249,7 +248,11 @@ public <R> R processRequest(
         final long start = registry.clock().wallTime();
         final Map<String, String> tags = Maps.newHashMap();
         tags.put("request", resourceRequestName);
-        tags.put("caller", MetacatContextManager.getContext().getClientAppName());
+        String clientAppName =  MetacatContextManager.getContext().getClientAppName();
+        if (clientAppName == null) {
+            clientAppName = "UNKNOWN";
+        }
+        tags.put("caller", clientAppName);
         registry.counter(requestCounterId.withTags(tags)).increment();
         try {
             MetacatContextManager.getContext().setRequestName(resourceRequestName);