chore: ignore null filters and issue warning message (#375)

tyroneyu · web-flow · commit ab163de63a74 · 2022-11-11T15:21:18.000-08:00
diff --git a/filter-api/src/main/java/org/hypertrace/agent/filter/FilterRegistry.java b/filter-api/src/main/java/org/hypertrace/agent/filter/FilterRegistry.java
@@ -87,7 +87,11 @@ private static Filter load(List<String> jarPaths) {
         continue;
       }
       Filter filter = provider.create();
-      filters.add(filter);
+      if (filter == null) {
+        logger.warn(String.format("%s returned null filter.", provider.getClass().getSimpleName()));
+      } else {
+        filters.add(filter);
+      }
     }
     return new MultiFilter(filters);
   }

Original file line number	Diff line number	Diff line change
`@@ -87,7 +87,11 @@ private static Filter load(List<String> jarPaths) {`
`87`	`87`	`continue;`
`88`	`88`	`}`
`89`	`89`	`Filter filter = provider.create();`
`90`		`- filters.add(filter);`
	`90`	`+ if (filter == null) {`
	`91`	`+ logger.warn(String.format("%s returned null filter.", provider.getClass().getSimpleName()));`
	`92`	`+ } else {`
	`93`	`+ filters.add(filter);`
	`94`	`+ }`
`91`	`95`	`}`
`92`	`96`	`return new MultiFilter(filters);`
`93`	`97`	`}`