merged main

added feedback message truncation
SentryClient doesn't check for backfillable feedbacks anymore

Author: stefanosiano

CHANGELOG.md

@@ -2,6 +2,11 @@
 
 ## Unreleased
 
+### Features
+
+- Add new User Feedback API ([#4286](https://github.com/getsentry/sentry-java/pull/4286))
+    - We now introduced Sentry.captureFeedback, which supersedes Sentry.captureUserFeedback
+
 ### Dependencies
 
 - Bump Gradle from v8.13 to v8.14.0 ([#4360](https://github.com/getsentry/sentry-java/pull/4360))
@@ -50,8 +55,6 @@
 
 ### Features
 
-- Add new User Feedback API ([#4286](https://github.com/getsentry/sentry-java/pull/4286))
-  - We now introduced Sentry.captureFeedback, which supersedes Sentry.captureUserFeedback
 - Add `SentryWrapper.wrapRunnable` to wrap `Runnable` for use with Sentry ([#4332](https://github.com/getsentry/sentry-java/pull/4332))
 
 ### Fixes

sentry/src/main/java/io/sentry/SentryClient.java

@@ -1048,10 +1048,8 @@ public void captureSession(final @NotNull Session session, final @Nullable Hint
       sentryId = event.getEventId();
     }
 
-    final boolean isBackfillable = HintUtils.hasType(hint, Backfillable.class);
-    // if event is backfillable we don't wanna trigger capture replay, because it's an event from
-    // the past. If feedback already has a replayId, we don't want to overwrite it.
-    if (feedback.getReplayId() == null && !isBackfillable) {
+    // If feedback already has a replayId, we don't want to overwrite it.
+    if (feedback.getReplayId() == null) {
       options.getReplayController().captureReplay(false);
       final @NotNull SentryId replayId = scope.getReplayId();
       if (!replayId.equals(SentryId.EMPTY_ID)) {

sentry/src/main/java/io/sentry/protocol/Feedback.java

@@ -31,7 +31,7 @@ public final class Feedback implements JsonUnknown, JsonSerializable {
   private @Nullable Map<String, Object> unknown;
 
   public Feedback(final @NotNull String message) {
-    this.message = message;
+    setMessage(message);
   }
 
   public Feedback(final @NotNull Feedback feedback) {
@@ -89,7 +89,12 @@ public void setUrl(final @Nullable String url) {
   }
 
   public void setMessage(final @NotNull String message) {
-    this.message = message;
+    // Sentry limits the message to 4096 characters
+    if (message.length() > 4096) {
+      this.message = message.substring(0, 4096);
+    } else {
+      this.message = message;
+    }
   }
 
   @Override

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

@@ -3166,19 +3166,6 @@ class SentryClientTest {
         )
     }
 
-    @Test
-    fun `captureFeedback does not capture replay when backfilled`() {
-        val replayController = mock<ReplayController>()
-        val sut = fixture.getSut { it.setReplayController(replayController) }
-
-        sut.captureFeedback(
-            Feedback("message"),
-            HintUtils.createWithTypeCheckHint(BackfillableHint()),
-            createScope()
-        )
-        verify(replayController, never()).captureReplay(any())
-    }
-
     //endregion
 
     private fun givenScopeWithStartedSession(errored: Boolean = false, crashed: Boolean = false): IScope {

sentry/src/test/java/io/sentry/protocol/FeedbackTest.kt

@@ -49,4 +49,13 @@ class FeedbackTest {
             assertEquals("unknown", it["unknown"])
         }
     }
+
+    @Test
+    fun `setting a message longer than 4096 characters truncates the message`() {
+        val feedback = fixture.getSut()
+        feedback.message = "X".repeat(4095) + "Y" + "Z"
+        val expectedMessage = "X".repeat(4095) + "Y"
+        assertEquals(expectedMessage, feedback.message)
+        assertEquals(4096, feedback.message.length)
+    }
 }