Merge branch 'main' into lcian/feat/ktor-client

Author: lcian

.craft.yml

@@ -6,8 +6,8 @@ targets:
     gradleCliPath: ./gradlew
     mavenCliPath: scripts/mvnw
     mavenSettingsPath: scripts/settings.xml
-    mavenRepoId: ossrh
-    mavenRepoUrl: https://oss.sonatype.org/service/local/staging/deploy/maven2/
+    mavenRepoId: ossrh-staging-api
+    mavenRepoUrl: https://ossrh-staging-api.central.sonatype.com/service/local/staging/deploy/maven2/
     android:
       distDirRegex: /^(sentry-android-|.*-android).*$/
       fileReplaceeRegex: /\d+\.\d+\.\d+(-\w+(\.\d+)?)?(-SNAPSHOT)?/

.github/workflows/codeql-analysis.yml

@@ -36,7 +36,7 @@ jobs:
           cache-encryption-key: ${{ secrets.GRADLE_ENCRYPTION_KEY }}
 
       - name: Initialize CodeQL
-        uses: github/codeql-action/init@ce28f5bb42b7a9f2c824e633a3f6ee835bab6858 # pin@v2
+        uses: github/codeql-action/init@39edc492dbe16b1465b0cafca41432d857bdb31a # pin@v2
         with:
           languages: 'java'
 
@@ -45,4 +45,4 @@ jobs:
           ./gradlew buildForCodeQL --no-build-cache
 
       - name: Perform CodeQL Analysis
-        uses: github/codeql-action/analyze@ce28f5bb42b7a9f2c824e633a3f6ee835bab6858 # pin@v2
+        uses: github/codeql-action/analyze@39edc492dbe16b1465b0cafca41432d857bdb31a # pin@v2

CHANGELOG.md

@@ -1,5 +1,58 @@
 # Changelog
 
+## Unreleased
+
+### Fixes
+
+- Optimize scope when maxBreadcrumb is 0 ([#4504](https://github.com/getsentry/sentry-java/pull/4504))
+- Fix javadoc on TransportResult ([#4528](https://github.com/getsentry/sentry-java/pull/4528))
+
+### Internal
+
+- Flattened PerformanceCollectionData ([#4505](https://github.com/getsentry/sentry-java/pull/4505))
+
+## 8.16.0
+
+### Features
+
+- Send JUL logs to Sentry as logs ([#4518](https://github.com/getsentry/sentry-java/pull/4518))
+  - You need to enable the logs feature, either in `sentry.properties`:
+    ```properties
+    logs.enabled=true
+    ```
+  - Or, if you manually initialize Sentry, you may also enable logs on `Sentry.init`:
+    ```java
+    Sentry.init(options -> {
+      ...
+      options.getLogs().setEnabled(true);
+    });
+    ```
+  - It is also possible to set the `minimumLevel` in `logging.properties`, meaning any log message >= the configured level will be sent to Sentry and show up under Logs:
+    ```properties
+    io.sentry.jul.SentryHandler.minimumLevel=CONFIG
+    ```
+- Send Log4j2 logs to Sentry as logs ([#4517](https://github.com/getsentry/sentry-java/pull/4517))
+  - You need to enable the logs feature either in `sentry.properties`:
+    ```properties
+    logs.enabled=true
+    ```
+  - If you manually initialize Sentry, you may also enable logs on `Sentry.init`:
+    ```java
+    Sentry.init(options -> {
+      ...
+      options.getLogs().setEnabled(true);
+    });
+    ```
+  - It is also possible to set the `minimumLevel` in `log4j2.xml`, meaning any log message >= the configured level will be sent to Sentry and show up under Logs:
+    ```xml
+    <Sentry name="Sentry"
+        dsn="your DSN"
+        minimumBreadcrumbLevel="DEBUG"
+        minimumEventLevel="WARN"
+        minimumLevel="DEBUG"
+    />
+    ```
+
 ## 8.15.1
 
 ### Fixes

gradle.properties

@@ -11,7 +11,7 @@ org.jetbrains.dokka.experimental.gradle.pluginMode=V2Enabled
 android.useAndroidX=true
 
 # Release information
-versionName=8.15.1
+versionName=8.16.0
 
 # Override the SDK name on native crashes on Android
 sentryAndroidSdkName=sentry.native.android

scripts/settings.xml

@@ -4,7 +4,7 @@
                           https://maven.apache.org/xsd/settings-1.0.0.xsd">
   <servers>
     <server>
-      <id>ossrh</id>
+      <id>ossrh-staging-api</id>
       <username>${env.OSSRH_USERNAME}</username>
       <password>${env.OSSRH_PASSWORD}</password>
     </server>

sentry-android-core/src/main/java/io/sentry/android/core/AndroidCpuCollector.java

@@ -3,12 +3,10 @@
 import android.os.SystemClock;
 import android.system.Os;
 import android.system.OsConstants;
-import io.sentry.CpuCollectionData;
 import io.sentry.ILogger;
 import io.sentry.IPerformanceSnapshotCollector;
 import io.sentry.PerformanceCollectionData;
 import io.sentry.SentryLevel;
-import io.sentry.SentryNanotimeDate;
 import io.sentry.util.FileUtils;
 import io.sentry.util.Objects;
 import java.io.File;
@@ -72,11 +70,8 @@ public void collect(final @NotNull PerformanceCollectionData performanceCollecti
     // number from 0 to 100, so we are going to multiply it by 100
     final double cpuUsagePercentage = cpuNanosDiff / (double) realTimeNanosDiff;
 
-    CpuCollectionData cpuData =
-        new CpuCollectionData(
-            (cpuUsagePercentage / (double) numCores) * 100.0, new SentryNanotimeDate());
-
-    performanceCollectionData.addCpuData(cpuData);
+    performanceCollectionData.setCpuUsagePercentage(
+        (cpuUsagePercentage / (double) numCores) * 100.0);
   }
 
   /** Read the /proc/self/stat file and parses the result. */

sentry-android-core/src/main/java/io/sentry/android/core/AndroidMemoryCollector.java

@@ -2,9 +2,7 @@
 
 import android.os.Debug;
 import io.sentry.IPerformanceSnapshotCollector;
-import io.sentry.MemoryCollectionData;
 import io.sentry.PerformanceCollectionData;
-import io.sentry.SentryNanotimeDate;
 import org.jetbrains.annotations.ApiStatus;
 import org.jetbrains.annotations.NotNull;
 
@@ -18,8 +16,7 @@ public void setup() {}
   public void collect(final @NotNull PerformanceCollectionData performanceCollectionData) {
     long usedMemory = Runtime.getRuntime().totalMemory() - Runtime.getRuntime().freeMemory();
     long usedNativeMemory = Debug.getNativeHeapSize() - Debug.getNativeHeapFreeSize();
-    MemoryCollectionData memoryData =
-        new MemoryCollectionData(usedMemory, usedNativeMemory, new SentryNanotimeDate());
-    performanceCollectionData.addMemoryData(memoryData);
+    performanceCollectionData.setUsedHeapMemory(usedMemory);
+    performanceCollectionData.setUsedNativeMemory(usedNativeMemory);
   }
 }

sentry-android-core/src/main/java/io/sentry/android/core/AndroidProfiler.java

@@ -4,14 +4,11 @@
 import android.os.Debug;
 import android.os.Process;
 import android.os.SystemClock;
-import io.sentry.CpuCollectionData;
 import io.sentry.DateUtils;
 import io.sentry.ILogger;
 import io.sentry.ISentryExecutorService;
 import io.sentry.ISentryLifecycleToken;
-import io.sentry.MemoryCollectionData;
 import io.sentry.PerformanceCollectionData;
-import io.sentry.SentryDate;
 import io.sentry.SentryLevel;
 import io.sentry.SentryNanotimeDate;
 import io.sentry.SentryUUID;
@@ -156,7 +153,7 @@ public void onFrameMetricCollected(
                   // profileStartNanos is calculated through SystemClock.elapsedRealtimeNanos(),
                   // but frameEndNanos uses System.nanotime(), so we convert it to get the timestamp
                   // relative to profileStartNanos
-                  final SentryDate timestamp = new SentryNanotimeDate();
+                  final long timestampNanos = new SentryNanotimeDate().nanoTimestamp();
                   final long frameTimestampRelativeNanos =
                       frameEndNanos
                           - System.nanoTime()
@@ -171,17 +168,17 @@ public void onFrameMetricCollected(
                   if (isFrozen) {
                     frozenFrameRenderMeasurements.addLast(
                         new ProfileMeasurementValue(
-                            frameTimestampRelativeNanos, durationNanos, timestamp));
+                            frameTimestampRelativeNanos, durationNanos, timestampNanos));
                   } else if (isSlow) {
                     slowFrameRenderMeasurements.addLast(
                         new ProfileMeasurementValue(
-                            frameTimestampRelativeNanos, durationNanos, timestamp));
+                            frameTimestampRelativeNanos, durationNanos, timestampNanos));
                   }
                   if (refreshRate != lastRefreshRate) {
                     lastRefreshRate = refreshRate;
                     screenFrameRateMeasurements.addLast(
                         new ProfileMeasurementValue(
-                            frameTimestampRelativeNanos, refreshRate, timestamp));
+                            frameTimestampRelativeNanos, refreshRate, timestampNanos));
                   }
                 }
               });
@@ -318,32 +315,28 @@ private void putPerformanceCollectionDataInMeasurements(
           new ArrayDeque<>(performanceCollectionData.size());
 
       synchronized (performanceCollectionData) {
-        for (PerformanceCollectionData performanceData : performanceCollectionData) {
-          CpuCollectionData cpuData = performanceData.getCpuData();
-          MemoryCollectionData memoryData = performanceData.getMemoryData();
-          if (cpuData != null) {
+        for (final @NotNull PerformanceCollectionData data : performanceCollectionData) {
+          final long nanoTimestamp = data.getNanoTimestamp();
+          final long relativeStartNs = nanoTimestamp + timestampDiff;
+          final @Nullable Double cpuUsagePercentage = data.getCpuUsagePercentage();
+          final @Nullable Long usedHeapMemory = data.getUsedHeapMemory();
+          final @Nullable Long usedNativeMemory = data.getUsedNativeMemory();
+
+          if (cpuUsagePercentage != null) {
             cpuUsageMeasurements.add(
-                new ProfileMeasurementValue(
-                    cpuData.getTimestamp().nanoTimestamp() + timestampDiff,
-                    cpuData.getCpuUsagePercentage(),
-                    cpuData.getTimestamp()));
+                new ProfileMeasurementValue(relativeStartNs, cpuUsagePercentage, nanoTimestamp));
           }
-          if (memoryData != null && memoryData.getUsedHeapMemory() > -1) {
+          if (usedHeapMemory != null) {
             memoryUsageMeasurements.add(
-                new ProfileMeasurementValue(
-                    memoryData.getTimestamp().nanoTimestamp() + timestampDiff,
-                    memoryData.getUsedHeapMemory(),
-                    memoryData.getTimestamp()));
+                new ProfileMeasurementValue(relativeStartNs, usedHeapMemory, nanoTimestamp));
           }
-          if (memoryData != null && memoryData.getUsedNativeMemory() > -1) {
+          if (usedNativeMemory != null) {
             nativeMemoryUsageMeasurements.add(
-                new ProfileMeasurementValue(
-                    memoryData.getTimestamp().nanoTimestamp() + timestampDiff,
-                    memoryData.getUsedNativeMemory(),
-                    memoryData.getTimestamp()));
+                new ProfileMeasurementValue(relativeStartNs, usedNativeMemory, nanoTimestamp));
           }
         }
       }
+
       if (!cpuUsageMeasurements.isEmpty()) {
         measurementsMap.put(
             ProfileMeasurement.ID_CPU_USAGE,

sentry-android-core/src/test/java/io/sentry/android/core/AndroidContinuousProfilerTest.kt

@@ -5,17 +5,14 @@ import android.os.Build
 import androidx.test.core.app.ApplicationProvider
 import androidx.test.ext.junit.runners.AndroidJUnit4
 import io.sentry.CompositePerformanceCollector
-import io.sentry.CpuCollectionData
 import io.sentry.DataCategory
 import io.sentry.IConnectionStatusProvider
 import io.sentry.ILogger
 import io.sentry.IScopes
-import io.sentry.MemoryCollectionData
 import io.sentry.PerformanceCollectionData
 import io.sentry.ProfileLifecycle
 import io.sentry.Sentry
 import io.sentry.SentryLevel
-import io.sentry.SentryNanotimeDate
 import io.sentry.SentryTracer
 import io.sentry.TracesSampler
 import io.sentry.TransactionContext
@@ -427,10 +424,11 @@ class AndroidContinuousProfilerTest {
   @Test
   fun `profiler sends chunk with measurements`() {
     val performanceCollector = mock<CompositePerformanceCollector>()
-    val collectionData = PerformanceCollectionData()
+    val collectionData = PerformanceCollectionData(10)
 
-    collectionData.addMemoryData(MemoryCollectionData(2, 3, SentryNanotimeDate()))
-    collectionData.addCpuData(CpuCollectionData(3.0, SentryNanotimeDate()))
+    collectionData.usedHeapMemory = 2
+    collectionData.usedNativeMemory = 3
+    collectionData.cpuUsagePercentage = 3.0
     whenever(performanceCollector.stop(any<String>())).thenReturn(listOf(collectionData))
 
     fixture.options.compositePerformanceCollector = performanceCollector

sentry-android-core/src/test/java/io/sentry/android/core/AndroidCpuCollectorTest.kt

@@ -28,20 +28,19 @@ class AndroidCpuCollectorTest {
 
   @Test
   fun `collect works only after setup`() {
-    val data = PerformanceCollectionData()
+    val data = PerformanceCollectionData(10)
     fixture.getSut().collect(data)
-    assertNull(data.cpuData)
+    assertNull(data.cpuUsagePercentage)
   }
 
   @Test
   fun `when collect cpu is collected`() {
-    val data = PerformanceCollectionData()
+    val data = PerformanceCollectionData(10)
     val collector = fixture.getSut()
     collector.setup()
     collector.collect(data)
-    val cpuData = data.cpuData
+    val cpuData = data.cpuUsagePercentage
     assertNotNull(cpuData)
-    assertNotEquals(0.0, cpuData.cpuUsagePercentage)
-    assertNotEquals(0, cpuData.timestamp.nanoTimestamp())
+    assertNotEquals(0.0, cpuData)
   }
 }