catch the numberformatexception

Author: mkleene

sdk/src/main/java/io/opentdf/platform/sdk/AddressNormalizer.java

@@ -33,7 +33,7 @@ static String normalizeAddress(String urlString, boolean usePlaintext) {
                 // otherwise, we have a scheme but no host, so we assume the scheme is actually the host and the SSP is the port
                 try {
                     uri = new URI(scheme, null, uri.getScheme(), Integer.parseInt(uri.getSchemeSpecificPart()), null, null, null);
-                } catch (URISyntaxException e) {
+                } catch (URISyntaxException | NumberFormatException e) {
                     throw new SDKException("error trying to create URL for host and port[" + urlString + "]", e);
                 }
             }

sdk/src/test/java/io/opentdf/platform/sdk/AddressNormalizerTest.java

@@ -5,6 +5,7 @@
 
 import static io.opentdf.platform.sdk.AddressNormalizer.normalizeAddress;
 import static org.assertj.core.api.AssertionsForClassTypes.assertThat;
+import static org.junit.Assert.assertThrows;
 
 class AddressNormalizerTest {
 
@@ -26,4 +27,13 @@ void testAddressNormaliationWithInsecureHTTPClient() {
         assertThat(normalizeAddress("sftp://example.org", true)).isEqualTo("http://example.org:80");
         assertThat(normalizeAddress("keycloak.vm", true)).isEqualTo("http://keycloak.vm:80");
     }
+
+    @Test
+    void testAddressNormalizationWithInvalidPort() {
+        var thrown = assertThrows(SDKException.class, () -> normalizeAddress("example.org:notaport", true));
+        assertThat(thrown.getMessage()).contains("example.org:notaport");
+
+        thrown = assertThrows(SDKException.class, () -> normalizeAddress("http://example.org:notaport", true));
+        assertThat(thrown.getMessage()).contains("http://example.org:notaport");
+    }
 }