coverage

Author: mkleene

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

@@ -15,9 +15,9 @@ private AddressNormalizer(){
     }
 
     static String normalizeAddress(String urlString, boolean usePlaintext) {
-        final String scheme = usePlaintext ? "http" : "https";
         URI uri = getInitialUri(urlString);
 
+        final String scheme = usePlaintext ? "http" : "https";
         if (uri.getHost() == null) {
             // if there is no host and no scheme, then we assume the input is a hostname with no port or scheme
             if (uri.getScheme() == null) {

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

@@ -3,6 +3,8 @@
 
 import org.junit.jupiter.api.Test;
 
+import java.net.URISyntaxException;
+
 import static io.opentdf.platform.sdk.AddressNormalizer.normalizeAddress;
 import static org.assertj.core.api.AssertionsForClassTypes.assertThat;
 import static org.junit.jupiter.api.Assertions.assertThrows;
@@ -40,4 +42,10 @@ void testAddressNormalizationWithInvalidPort() {
         thrown = assertThrows(SDKException.class, () -> normalizeAddress("http://example.org:notaport", true));
         assertThat(thrown.getMessage()).contains("http://example.org:notaport");
     }
+
+    @Test
+    void testInsaneAddressThrowsException() {
+        var thrown = assertThrows(SDKException.class, () -> normalizeAddress("1://&()&{$!//1//1", true));
+        assertThat(thrown.getCause()).isInstanceOf(URISyntaxException.class);
+    }
 }