Remove buildSrc (#2379)

We don't use the upload results feature (kokoro picks the results
artifacts directly and uploads them).

Keeping it around is a maintenance burden.

Also fixed a deprecation warning.

prober/src/main/java/google/registry/monitoring/blackbox/handler/WebWhoisActionHandler.java

@@ -31,6 +31,8 @@ import io.netty.channel.ChannelFuture;
 import io.netty.channel.ChannelHandlerContext;
 import io.netty.handler.codec.http.HttpResponseStatus;
 import java.net.MalformedURLException;
+import java.net.URI;
+import java.net.URISyntaxException;
 import java.net.URL;
 import javax.inject.Inject;
 import org.joda.time.Duration;
@@ -99,8 +101,8 @@ public class WebWhoisActionHandler extends ActionHandler {
       // Obtain url to be redirected to
       URL url;
       try {
-        url = new URL(response.headers().get("Location"));
-      } catch (MalformedURLException e) {
+        url = new URI(response.headers().get("Location")).toURL();
+      } catch (MalformedURLException | URISyntaxException | IllegalArgumentException e) {
         // in case of error, log it, and let ActionHandler's exceptionThrown method deal with it
         throw new FailureException(
             "Redirected Location was invalid. Given Location was: "

prober/src/test/java/google/registry/monitoring/blackbox/handler/WebWhoisActionHandlerTest.java

@@ -172,7 +172,7 @@ class WebWhoisActionHandlerTest {
     assertThat(future.isSuccess()).isFalse();
 
     // Ensures that we fail as a result of a FailureException.
-    assertThat(future.cause() instanceof FailureException).isTrue();
+    assertThat(future.cause()).isInstanceOf(FailureException.class);
   }
 
   @Test