From 5293d25f555890d0af31caa4da185256161857d4 Mon Sep 17 00:00:00 2001
From: Andrew Azores <aazores@redhat.com>
Date: Fri, 22 Mar 2024 09:56:37 -0400
Subject: [PATCH] spotbugs

---
 src/main/java/io/cryostat/discovery/Discovery.java       | 5 ++---
 src/main/java/io/cryostat/discovery/DiscoveryPlugin.java | 2 ++
 2 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/src/main/java/io/cryostat/discovery/Discovery.java b/src/main/java/io/cryostat/discovery/Discovery.java
index ffc34667b..d4c6e4ced 100644
--- a/src/main/java/io/cryostat/discovery/Discovery.java
+++ b/src/main/java/io/cryostat/discovery/Discovery.java
@@ -33,7 +33,6 @@
 import java.util.Optional;
 import java.util.Set;
 import java.util.UUID;
-import java.util.regex.Pattern;
 
 import io.cryostat.credentials.Credential;
 import io.cryostat.discovery.DiscoveryPlugin.PluginCallback;
@@ -43,6 +42,7 @@
 import com.fasterxml.jackson.databind.ObjectMapper;
 import com.nimbusds.jose.JOSEException;
 import com.nimbusds.jwt.proc.BadJWTException;
+import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
 import io.quarkus.runtime.ShutdownEvent;
 import io.quarkus.runtime.StartupEvent;
 import io.smallrye.common.annotation.Blocking;
@@ -88,8 +88,6 @@
 @Path("")
 public class Discovery {
 
-    public static final Pattern HOST_PORT_PAIR_PATTERN =
-            Pattern.compile("^([^:\\s]+)(?::(\\d{1,5}))$");
     static final String X_FORWARDED_FOR = "X-Forwarded-For";
 
     private static final String JOB_PERIODIC = "periodic";
@@ -389,6 +387,7 @@ static class RefreshPluginJob implements Job {
         @Override
         @Transactional
         @Blocking
+        @SuppressFBWarnings("RCN_REDUNDANT_NULLCHECK_OF_NULL_VALUE")
         public void execute(JobExecutionContext context) throws JobExecutionException {
             DiscoveryPlugin plugin = null;
             try {
diff --git a/src/main/java/io/cryostat/discovery/DiscoveryPlugin.java b/src/main/java/io/cryostat/discovery/DiscoveryPlugin.java
index 4ca52cdeb..75eaf20b7 100644
--- a/src/main/java/io/cryostat/discovery/DiscoveryPlugin.java
+++ b/src/main/java/io/cryostat/discovery/DiscoveryPlugin.java
@@ -25,6 +25,7 @@
 import io.cryostat.credentials.Credential;
 
 import com.fasterxml.jackson.annotation.JsonProperty;
+import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
 import io.quarkus.hibernate.orm.panache.PanacheEntityBase;
 import io.quarkus.rest.client.reactive.QuarkusRestClientBuilder;
 import jakarta.enterprise.context.ApplicationScoped;
@@ -136,6 +137,7 @@ public static class DiscoveryPluginAuthorizationHeaderFactory
 
             private final DiscoveryPlugin plugin;
 
+            @SuppressFBWarnings("EI_EXPOSE_REP2")
             public DiscoveryPluginAuthorizationHeaderFactory(DiscoveryPlugin plugin) {
                 this.plugin = plugin;
             }