Merge pull request #1517 from IABTechLab/ans-UID2-4956-add-site-id-extraction-logic

ashleysmithTTD · web-flow · commit 58da2f670176 · 2025-02-27T17:53:43.000-07:00
if site id is null do not log
diff --git a/src/main/java/com/uid2/operator/vertx/UIDOperatorVerticle.java b/src/main/java/com/uid2/operator/vertx/UIDOperatorVerticle.java
@@ -896,11 +896,13 @@ private void handleTokenRefreshV1(RoutingContext rc) {
     }
 
     public void recordOperatorServedSdkUsage(Integer siteId, RoutingContext rc, String clientVersion) {
-        _clientVersionCounters.computeIfAbsent(new Tuple.Tuple2<>(Integer.toString(siteId), clientVersion), tuple -> Counter
+        if (siteId != 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")
                     .tags("site_id", tuple.getItem1(), "client_version", tuple.getItem2())
                     .register(Metrics.globalRegistry)).increment();;
+        }
     }
 
     private void handleTokenRefreshV2(RoutingContext rc) {

Original file line number	Diff line number	Diff line change
`@@ -896,11 +896,13 @@ private void handleTokenRefreshV1(RoutingContext rc) {`
`896`	`896`	`}`
`897`	`897`
`898`	`898`	`public void recordOperatorServedSdkUsage(Integer siteId, RoutingContext rc, String clientVersion) {`
`899`		`- _clientVersionCounters.computeIfAbsent(new Tuple.Tuple2<>(Integer.toString(siteId), clientVersion), tuple -> Counter`
	`899`	`+ if (siteId != null) {`
	`900`	`+ _clientVersionCounters.computeIfAbsent(new Tuple.Tuple2<>(Integer.toString(siteId), clientVersion), tuple -> Counter`
`900`	`901`	`.builder("uid2.client_sdk_versions")`
`901`	`902`	`.description("counter for how many http requests are processed per each operator-served sdk version")`
`902`	`903`	`.tags("site_id", tuple.getItem1(), "client_version", tuple.getItem2())`
`903`	`904`	`.register(Metrics.globalRegistry)).increment();;`
	`905`	`+ }`
`904`	`906`	`}`
`905`	`907`
`906`	`908`	`private void handleTokenRefreshV2(RoutingContext rc) {`