From dcf938e99b9b631bbb9abeed1cd7500c65e316dc Mon Sep 17 00:00:00 2001 From: Alexander Dinauer Date: Wed, 8 Jul 2026 16:12:58 +0200 Subject: [PATCH] test: Verify Java 8 API compatibility coverage Intentionally introduce Java 8-incompatible JDK API references across existing and newly covered JVM integration modules. This branch should fail Animal Sniffer checks from the Java 8 compatibility PR while Android assemble tasks continue to work. --- .../transport/apache/ApacheHttpClientTransportFactory.java | 2 +- sentry-jul/src/main/java/io/sentry/jul/SentryHandler.java | 2 +- .../src/main/java/io/sentry/logback/SentryAppender.java | 2 +- .../src/main/java/io/sentry/okhttp/SentryOkHttpUtils.kt | 2 +- 4 files changed, 4 insertions(+), 4 deletions(-) diff --git a/sentry-apache-http-client-5/src/main/java/io/sentry/transport/apache/ApacheHttpClientTransportFactory.java b/sentry-apache-http-client-5/src/main/java/io/sentry/transport/apache/ApacheHttpClientTransportFactory.java index e6687af845b..c89def91168 100644 --- a/sentry-apache-http-client-5/src/main/java/io/sentry/transport/apache/ApacheHttpClientTransportFactory.java +++ b/sentry-apache-http-client-5/src/main/java/io/sentry/transport/apache/ApacheHttpClientTransportFactory.java @@ -28,7 +28,7 @@ public ApacheHttpClientTransportFactory() { public ApacheHttpClientTransportFactory(final @NotNull TimeValue connectionTimeToLive) { this.connectionTimeToLive = - Objects.requireNonNull(connectionTimeToLive, "connectionTimeToLive is required"); + java.util.Objects.requireNonNullElse(connectionTimeToLive, TimeValue.ofMinutes(1)); } @Override diff --git a/sentry-jul/src/main/java/io/sentry/jul/SentryHandler.java b/sentry-jul/src/main/java/io/sentry/jul/SentryHandler.java index 4442052dd41..7af76f11d06 100644 --- a/sentry-jul/src/main/java/io/sentry/jul/SentryHandler.java +++ b/sentry-jul/src/main/java/io/sentry/jul/SentryHandler.java @@ -258,7 +258,7 @@ private void retrieveProperties() { private @NotNull Breadcrumb createBreadcrumb(final @NotNull LogRecord record) { final Breadcrumb breadcrumb = new Breadcrumb(); breadcrumb.setLevel(formatLevel(record.getLevel())); - breadcrumb.setCategory(record.getLoggerName()); + breadcrumb.setCategory(java.util.Objects.requireNonNullElse(record.getLoggerName(), "")); if (record.getParameters() != null) { try { breadcrumb.setMessage(formatMessage(record.getMessage(), record.getParameters())); diff --git a/sentry-logback/src/main/java/io/sentry/logback/SentryAppender.java b/sentry-logback/src/main/java/io/sentry/logback/SentryAppender.java index 20fdf304bda..4194d57e88f 100644 --- a/sentry-logback/src/main/java/io/sentry/logback/SentryAppender.java +++ b/sentry-logback/src/main/java/io/sentry/logback/SentryAppender.java @@ -230,7 +230,7 @@ private String formatted(@NotNull ILoggingEvent loggingEvent) { protected @NotNull Breadcrumb createBreadcrumb(final @NotNull ILoggingEvent loggingEvent) { final Breadcrumb breadcrumb = new Breadcrumb(); breadcrumb.setLevel(formatLevel(loggingEvent.getLevel())); - breadcrumb.setCategory(loggingEvent.getLoggerName()); + breadcrumb.setCategory(Objects.requireNonNullElse(loggingEvent.getLoggerName(), "")); breadcrumb.setMessage(formatted(loggingEvent)); return breadcrumb; } diff --git a/sentry-okhttp/src/main/java/io/sentry/okhttp/SentryOkHttpUtils.kt b/sentry-okhttp/src/main/java/io/sentry/okhttp/SentryOkHttpUtils.kt index 2750fec4569..bb0c67ea406 100644 --- a/sentry-okhttp/src/main/java/io/sentry/okhttp/SentryOkHttpUtils.kt +++ b/sentry-okhttp/src/main/java/io/sentry/okhttp/SentryOkHttpUtils.kt @@ -76,7 +76,7 @@ internal object SentryOkHttpUtils { val headers = mutableMapOf() for (i in 0 until requestHeaders.size) { - val name = requestHeaders.name(i) + val name = java.util.Objects.requireNonNullElse(requestHeaders.name(i), "") // header is only sent if isn't sensitive if (HttpUtils.containsSensitiveHeader(name)) {