return null instead of []

IABTechLab · Oct 5, 2023 · 902cbfa · 902cbfa
1 parent 4d5d988
commit 902cbfa
Showing 1 changed file with 2 additions and 12 deletions.
diff --git a/src/main/java/com/uid2/admin/vertx/service/SharingService.java b/src/main/java/com/uid2/admin/vertx/service/SharingService.java
@@ -232,13 +232,8 @@ private void handleListAllowedSites(RoutingContext rc) {
             return;
         }
 
-        JsonArray listedSites = new JsonArray();
-        Set<Integer> allowedSites = keyset.getAllowedSites();
-        if (allowedSites != null) {
-            allowedSites.stream().sorted().forEach((listedSiteId) -> listedSites.add(listedSiteId));
-        }
         JsonObject jo = new JsonObject();
-        jo.put("allowed_sites", listedSites);
+        jo.put("allowed_sites", keyset.getAllowedSites());
         jo.put("allowed_types", keyset.getAllowedTypes());
         jo.put("hash", keyset.hashCode());
 
@@ -252,15 +247,10 @@ private void handleListAllAllowedSites(RoutingContext rc) {
             JsonArray ja = new JsonArray();
             Map<Integer, AdminKeyset> collection = this.keysetProvider.getSnapshot().getAllKeysets();
             for (Map.Entry<Integer, AdminKeyset> keyset : collection.entrySet()) {
-                JsonArray listedSites = new JsonArray();
-                Set<Integer> allowedSites = keyset.getValue().getAllowedSites();
-                if (allowedSites != null) {
-                    allowedSites.stream().sorted().forEach((listedSiteId) -> listedSites.add(listedSiteId));
-                }
                 JsonObject jo = new JsonObject();
                 jo.put("keyset_id", keyset.getValue().getKeysetId());
                 jo.put("site_id", keyset.getValue().getSiteId());
-                jo.put("allowed_sites", listedSites);
+                jo.put("allowed_sites", keyset.getValue().getAllowedSites());
                 jo.put("allowed_types", keyset.getValue().getAllowedTypes());
                 jo.put("hash", keyset.getValue().hashCode());
                 ja.add(jo);