fix(sentry): Mark skipped values consumed after success

adinauer · claude · adinauer · commit 1bcb316302e5 · 2026-04-16T15:02:45.000+02:00
Move skipValue consumption tracking after the underlying Gson skip succeeds.

This keeps valueConsumed aligned with the primitive reader methods and avoids marking a value as consumed before JsonReader has actually advanced past it.

Co-Authored-By: Claude &lt;noreply@anthropic.com&gt;
diff --git a/sentry/src/main/java/io/sentry/JsonObjectReader.java b/sentry/src/main/java/io/sentry/JsonObjectReader.java
@@ -349,8 +349,8 @@ public void setLenient(boolean lenient) {
 
   @Override
   public void skipValue() throws IOException {
-    markValueConsumed();
     jsonReader.skipValue();
+    markValueConsumed();
   }
 
   private boolean recoverAfterValueFailure(

Original file line number	Diff line number	Diff line change
`@@ -349,8 +349,8 @@ public void setLenient(boolean lenient) {`
`349`	`349`
`350`	`350`	`@Override`
`351`	`351`	`public void skipValue() throws IOException {`
`352`		`- markValueConsumed();`
`353`	`352`	`jsonReader.skipValue();`
	`353`	`+ markValueConsumed();`
`354`	`354`	`}`
`355`	`355`
`356`	`356`	`private boolean recoverAfterValueFailure(`