Minor code cleanup.

2 files changed, 5 insertions, 7 deletions

diff --git a/http-utils/src/main/java/ai/vespa/util/http/hc5/HttpToHttpsRoutePlanner.java b/http-utils/src/main/java/ai/vespa/util/http/hc5/HttpToHttpsRoutePlanner.java
index 962e6b32947..0a22437f5fc 100644
--- a/http-utils/src/main/java/ai/vespa/util/http/hc5/HttpToHttpsRoutePlanner.java
+++ b/http-utils/src/main/java/ai/vespa/util/http/hc5/HttpToHttpsRoutePlanner.java
@@ -4,7 +4,6 @@ package ai.vespa.util.http.hc5;
 import org.apache.hc.client5.http.HttpRoute;
 import org.apache.hc.client5.http.protocol.HttpClientContext;
 import org.apache.hc.client5.http.routing.HttpRoutePlanner;
-import org.apache.hc.core5.http.HttpException;
 import org.apache.hc.core5.http.HttpHost;
 import org.apache.hc.core5.http.protocol.HttpContext;
 
@@ -17,7 +16,7 @@ import org.apache.hc.core5.http.protocol.HttpContext;
 class HttpToHttpsRoutePlanner implements HttpRoutePlanner {
 
     @Override
-    public HttpRoute determineRoute(HttpHost target, HttpContext context) throws HttpException {
+    public HttpRoute determineRoute(HttpHost target, HttpContext context) {
         if ( ! target.getSchemeName().equals("http") && ! target.getSchemeName().equals("https"))
             throw new IllegalArgumentException("Scheme must be 'http' or 'https' when using HttpToHttpsRoutePlanner, was '" + target.getSchemeName() + "'");
 
diff --git a/http-utils/src/test/java/ai/vespa/util/http/hc5/HttpToHttpsRoutePlannerTest.java b/http-utils/src/test/java/ai/vespa/util/http/hc5/HttpToHttpsRoutePlannerTest.java
index b20d801e39c..6dbfb0a2312 100644
--- a/http-utils/src/test/java/ai/vespa/util/http/hc5/HttpToHttpsRoutePlannerTest.java
+++ b/http-utils/src/test/java/ai/vespa/util/http/hc5/HttpToHttpsRoutePlannerTest.java
@@ -4,7 +4,6 @@ package ai.vespa.util.http.hc5;
 import org.apache.hc.client5.http.HttpRoute;
 import org.apache.hc.client5.http.config.RequestConfig;
 import org.apache.hc.client5.http.protocol.HttpClientContext;
-import org.apache.hc.core5.http.HttpException;
 import org.apache.hc.core5.http.HttpHost;
 import org.junit.jupiter.api.Test;
 
@@ -18,7 +17,7 @@ public class HttpToHttpsRoutePlannerTest {
     final HttpToHttpsRoutePlanner planner = new HttpToHttpsRoutePlanner();
 
     @Test
-    void verifySchemeMustBeHttp() throws HttpException {
+    void verifySchemeMustBeHttp() {
         try {
             planner.determineRoute(new HttpHost("https", "host", 1), new HttpClientContext());
         }
@@ -28,7 +27,7 @@ public class HttpToHttpsRoutePlannerTest {
     }
 
     @Test
-    void verifyPortMustBeSet() throws HttpException {
+    void verifyPortMustBeSet() {
         try {
             planner.determineRoute(new HttpHost("http", "host", -1), new HttpClientContext());
         }
@@ -39,7 +38,7 @@ public class HttpToHttpsRoutePlannerTest {
 
 
     @Test
-    void verifyProxyIsDisallowed() throws HttpException {
+    void verifyProxyIsDisallowed() {
         HttpClientContext context = new HttpClientContext();
         context.setRequestConfig(RequestConfig.custom().setProxy(new HttpHost("proxy")).build());
         try {
@@ -51,7 +50,7 @@ public class HttpToHttpsRoutePlannerTest {
     }
 
     @Test
-    void verifySchemeIsRewritten() throws HttpException {
+    void verifySchemeIsRewritten() {
         assertEquals(new HttpRoute(new HttpHost("https", "host", 1)),
                 planner.determineRoute(new HttpHost("http", "host", 1), new HttpClientContext()));
     }