Merge pull request #1520 from IABTechLab/vse-UID2-4968-fix-operator-main

vishalegbert-ttd · web-flow · commit 7fa771571cc9 · 2025-02-28T16:05:14.000+11:00
Fix NPE in client version metrics
diff --git a/src/main/java/com/uid2/operator/vertx/UIDOperatorVerticle.java b/src/main/java/com/uid2/operator/vertx/UIDOperatorVerticle.java
@@ -896,7 +896,7 @@ private void handleTokenRefreshV1(RoutingContext rc) {
     }
 
     public void recordOperatorServedSdkUsage(Integer siteId, RoutingContext rc, String clientVersion) {
-        if (siteId != null) {
+        if (siteId != null && clientVersion != null) {
             _clientVersionCounters.computeIfAbsent(new Tuple.Tuple2<>(Integer.toString(siteId), clientVersion), tuple -> Counter
                     .builder("uid2.client_sdk_versions")
                     .description("counter for how many http requests are processed per each operator-served sdk version")

Original file line number	Diff line number	Diff line change
`@@ -896,7 +896,7 @@ private void handleTokenRefreshV1(RoutingContext rc) {`
`896`	`896`	`}`
`897`	`897`
`898`	`898`	`public void recordOperatorServedSdkUsage(Integer siteId, RoutingContext rc, String clientVersion) {`
`899`		`- if (siteId != null) {`
	`899`	`+ if (siteId != null && clientVersion != null) {`
`900`	`900`	`_clientVersionCounters.computeIfAbsent(new Tuple.Tuple2<>(Integer.toString(siteId), clientVersion), tuple -> Counter`
`901`	`901`	`.builder("uid2.client_sdk_versions")`
`902`	`902`	`.description("counter for how many http requests are processed per each operator-served sdk version")`