fix(dsn): Give a clear error message for a malformed port

Parse the port in a dedicated helper that reports the offending value
("Invalid DSN: Invalid port 'abc'.") instead of leaking the raw
NumberFormatException text. Narrow the catch to URISyntaxException now
that the port is the only parseInt, and add a test.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

Author: runningcode

sentry/src/main/java/io/sentry/Dsn.java

@@ -88,7 +88,7 @@ URI getSentryUri() {
       final String hostPort = hostAndPath.substring(0, firstSlash);
       final int portColon = portSeparatorIndex(hostPort);
       final String host = portColon < 0 ? hostPort : hostPort.substring(0, portColon);
-      final int port = portColon < 0 ? -1 : Integer.parseInt(hostPort.substring(portColon + 1));
+      final int port = portColon < 0 ? -1 : parsePort(hostPort.substring(portColon + 1));
 
       final String rawPath = stripTrailingSlash(collapseSlashes(hostAndPath.substring(firstSlash)));
       final int projectIdStart = rawPath.lastIndexOf('/') + 1;
@@ -100,11 +100,19 @@ URI getSentryUri() {
 
       sentryUri = new URI(scheme, null, host, port, path + "api/" + projectId, null, null);
       orgId = extractOrgId(host);
-    } catch (URISyntaxException | NumberFormatException e) {
+    } catch (URISyntaxException e) {
       throw new IllegalArgumentException("Invalid DSN: " + e.getMessage(), e);
     }
   }
 
+  private static int parsePort(final @NotNull String portString) {
+    try {
+      return Integer.parseInt(portString);
+    } catch (NumberFormatException e) {
+      throw new IllegalArgumentException("Invalid DSN: Invalid port '" + portString + "'.", e);
+    }
+  }
+
   // Drops the query string and/or fragment, whichever appears first, from the host onwards.
   private static @NotNull String stripQueryAndFragment(
       final @NotNull String dsn, final int fromIndex) {

sentry/src/test/java/io/sentry/DsnTest.kt

@@ -159,6 +159,12 @@ class DsnTest {
     assertThat(ex).hasMessageThat().isEqualTo("Invalid DSN: A Project Id is required.")
   }
 
+  @Test
+  fun `when port is not a number, throws exception`() {
+    val ex = assertFailsWith<IllegalArgumentException> { Dsn("http://key@host:abc/1") }
+    assertThat(ex).hasMessageThat().isEqualTo("Invalid DSN: Invalid port 'abc'.")
+  }
+
   @Test
   fun `dsn parsed with multiple path segments`() {
     val dsn = Dsn("https://key@host/path/to/sentry/id")