fix: Trim DSN string before URI parsing (#5113)

* fix: Trim DSN string before URI parsing

Trailing or leading whitespace in the DSN string (commonly introduced
by copy-paste) causes a URISyntaxException that crashes the application
on startup. Trim the DSN before passing it to the URI constructor.

Fixes GH-5087
Co-Authored-By: Claude <noreply@anthropic.com>

* docs: Add changelog entry for DSN trimming fix

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

* also trim on SentryOptions.setDsn

* fix: Throw clear error when DSN is empty

Previously an empty or whitespace-only DSN string would fall through
to the URI constructor, producing a confusing error message. Now the
Dsn constructor checks for empty strings after trimming and throws
an IllegalArgumentException with a clear message.

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

* Format code

* ci: retrigger checks

---------

Co-authored-by: Claude <noreply@anthropic.com>
Co-authored-by: Sentry Github Bot <bot+github-bot@sentry.io>

Author: 3 people

CHANGELOG.md

@@ -31,6 +31,7 @@
 ### Fixes
 
 - Fix crash when unregistering `SystemEventsBroadcastReceiver` with try-catch block. ([#5106](https://github.com/getsentry/sentry-java/pull/5106))
+- Trim DSN string before parsing to avoid `URISyntaxException` caused by trailing whitespace ([#5113](https://github.com/getsentry/sentry-java/pull/5113))
 
 ### Dependencies
 

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

@@ -50,8 +50,11 @@ URI getSentryUri() {
 
   Dsn(@Nullable String dsn) throws IllegalArgumentException {
     try {
-      Objects.requireNonNull(dsn, "The DSN is required.");
-      final URI uri = new URI(dsn).normalize();
+      final String dsnString = Objects.requireNonNull(dsn, "The DSN is required.").trim();
+      if (dsnString.isEmpty()) {
+        throw new IllegalArgumentException("The DSN is empty.");
+      }
+      final URI uri = new URI(dsnString).normalize();
       final String scheme = uri.getScheme();
       if (!("http".equalsIgnoreCase(scheme) || "https".equalsIgnoreCase(scheme))) {
         throw new IllegalArgumentException("Invalid DSN scheme: " + scheme);

sentry/src/main/java/io/sentry/SentryOptions.java

@@ -749,7 +749,7 @@ Dsn retrieveParsedDsn() throws IllegalArgumentException {
    * @param dsn the DSN
    */
   public void setDsn(final @Nullable String dsn) {
-    this.dsn = dsn;
+    this.dsn = dsn != null ? dsn.trim() : null;
     this.parsedDsn.resetValue();
 
     dsnHash = StringUtils.calculateStringHash(this.dsn, logger);

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

@@ -89,6 +89,24 @@ class DsnTest {
     assertEquals("http://host/api/id", dsn.sentryUri.toURL().toString())
   }
 
+  @Test
+  fun `dsn parsed with leading and trailing whitespace`() {
+    val dsn = Dsn("  https://key@host/id  ")
+    assertEquals("https://host/api/id", dsn.sentryUri.toURL().toString())
+  }
+
+  @Test
+  fun `when dsn is empty, throws exception`() {
+    val ex = assertFailsWith<IllegalArgumentException> { Dsn("") }
+    assertEquals("java.lang.IllegalArgumentException: The DSN is empty.", ex.message)
+  }
+
+  @Test
+  fun `when dsn is only whitespace, throws exception`() {
+    val ex = assertFailsWith<IllegalArgumentException> { Dsn("   ") }
+    assertEquals("java.lang.IllegalArgumentException: The DSN is empty.", ex.message)
+  }
+
   @Test
   fun `non http protocols are not accepted`() {
     assertFailsWith<IllegalArgumentException> { Dsn("ftp://publicKey:secretKey@host/path/id") }

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

@@ -588,6 +588,24 @@ class SentryOptionsTest {
     assertFalse(cacheDirPathWithoutDsn.contains(hash.toString()))
   }
 
+  @Test
+  fun `when setting dsn with whitespace, it is trimmed and produces the same cache dir path`() {
+    val dsn = "http://key@localhost/proj"
+    val options1 =
+      SentryOptions().apply {
+        setDsn(dsn)
+        cacheDirPath = "${File.separator}test"
+      }
+    val options2 =
+      SentryOptions().apply {
+        setDsn("  $dsn  ")
+        cacheDirPath = "${File.separator}test"
+      }
+
+    assertEquals(dsn, options2.dsn)
+    assertEquals(options1.cacheDirPath, options2.cacheDirPath)
+  }
+
   @Test
   fun `when options are initialized, idleTimeout is 3000`() {
     assertEquals(3000L, SentryOptions().idleTimeout)