Merge pull request #1107 from amvanbaren/debug-azure-user-agent

Check that userAgentHeader is set
eclipse · Feb 13, 2025 · 0fe75f6 · 0fe75f6
2 parents dde96a5 + 76b1c50
commit 0fe75f6
Showing 1 changed file with 2 additions and 6 deletions.
diff --git a/server/src/main/java/org/eclipse/openvsx/storage/AzureDownloadCountService.java b/server/src/main/java/org/eclipse/openvsx/storage/AzureDownloadCountService.java
@@ -228,13 +228,9 @@ private boolean isExtensionPackageUri(JsonNode node) {
     private boolean isNotOpenVSXUserAgent(JsonNode node) {
         var userAgentHeader = node.path("properties").path("userAgentHeader").asText();
         if(StringUtils.isEmpty(userAgentHeader)) {
-            try {
-                logger.info("USER AGENT HEADER IS EMPTY, JSON:\n{}", getObjectMapper().writeValueAsString(node));
-            } catch (JsonProcessingException e) {
-                // do nothing, this logger statement is for debugging purposes
-            }
-
             throw new IllegalArgumentException("Expected node to have userAgentHeader property");
+        } else if (userAgentHeader.equals(AZURE_USER_AGENT)) {
+            logger.info("userAgentHeader: {}", userAgentHeader);
         }
 
         return !userAgentHeader.equals(AZURE_USER_AGENT);