revert jul

Author: lcian

sentry-jul/src/main/java/io/sentry/jul/SentryHandler.java

@@ -154,16 +154,9 @@ protected void captureLog(@NotNull LogRecord loggingEvent) {
       message = loggingEvent.getResourceBundle().getString(loggingEvent.getMessage());
     }
 
-    String formattedMessage = null;
-    boolean formattingFailed = false;
-    try {
-      formattedMessage = maybeFormatted(arguments, message);
-    } catch (RuntimeException e) {
-      formattedMessage = message;
-      formattingFailed = true;
-    }
+    final @NotNull String formattedMessage = maybeFormatted(arguments, message);
 
-    if (formattingFailed || !formattedMessage.equals(message)) {
+    if (!formattedMessage.equals(message)) {
       attributes.add(SentryAttribute.stringAttribute("sentry.message.template", message));
     }
 
@@ -180,7 +173,6 @@ protected void captureLog(@NotNull LogRecord loggingEvent) {
         return formatMessage(message, arguments);
       } catch (RuntimeException e) {
         // local formatting failed, sending raw message instead of formatted message
-        throw e;
       }
     }
 

sentry-jul/src/test/kotlin/io/sentry/jul/SentryHandlerTest.kt

@@ -555,22 +555,4 @@ class SentryHandlerTest {
         }
       )
   }
-
-  @Test
-  fun `sets template on log when logging message with parameters and formatting fails due to 0 args`() {
-    fixture = Fixture(minimumLevel = Level.SEVERE, printfStyle = true)
-    fixture.logger.log(Level.SEVERE, "testing message %d", emptyArray())
-
-    Sentry.flush(1000)
-
-    verify(fixture.transport)
-      .send(
-        checkLogs { logs ->
-          val log = logs.items.first()
-          assertEquals("testing message %d", log.body)
-          assertEquals("testing message %d", log.attributes?.get("sentry.message.template")?.value)
-          assertNull(log.attributes?.get("sentry.message.parameter.0"))
-        }
-      )
-  }
 }