Fix whitespace-only DSN handling

cursoragent · cursoragent · commit 0a7e3bd9465b · 2026-03-02T11:40:31.000Z
diff --git a/sentry/src/main/java/io/sentry/SentryOptions.java b/sentry/src/main/java/io/sentry/SentryOptions.java
@@ -749,7 +749,12 @@ Dsn retrieveParsedDsn() throws IllegalArgumentException {
    * @param dsn the DSN
    */
   public void setDsn(final @Nullable String dsn) {
-    this.dsn = dsn != null ? dsn.trim() : null;
+    if (dsn == null) {
+      this.dsn = null;
+    } else {
+      final String trimmedDsn = dsn.trim();
+      this.dsn = trimmedDsn.isEmpty() && !dsn.isEmpty() ? dsn : trimmedDsn;
+    }
     this.parsedDsn.resetValue();
 
     dsnHash = StringUtils.calculateStringHash(this.dsn, logger);
diff --git a/sentry/src/test/java/io/sentry/SentryTest.kt b/sentry/src/test/java/io/sentry/SentryTest.kt
@@ -340,6 +340,11 @@ class SentryTest {
     )
   }
 
+  @Test
+  fun `initializes Sentry with whitespace-only dsn, throwing IllegalArgumentException`() {
+    assertThrows(java.lang.IllegalArgumentException::class.java) { initForTest { it.dsn = "   " } }
+  }
+
   @Test
   fun `captureUserFeedback gets forwarded to client`() {
     initForTest { it.dsn = dsn }

Original file line number	Diff line number	Diff line change
`@@ -340,6 +340,11 @@ class SentryTest {`
`340`	`340`	`)`
`341`	`341`	`}`
`342`	`342`
	`343`	`+ @Test`
	`344`	+ fun `initializes Sentry with whitespace-only dsn, throwing IllegalArgumentException`() {
	`345`	`+ assertThrows(java.lang.IllegalArgumentException::class.java) { initForTest { it.dsn = " " } }`
	`346`	`+ }`
	`347`	`+`
`343`	`348`	`@Test`
`344`	`349`	fun `captureUserFeedback gets forwarded to client`() {
`345`	`350`	`initForTest { it.dsn = dsn }`