Merge branch '2.19'

Author: cowtowncoder

ion/src/main/java/tools/jackson/dataformat/ion/IonParser.java

@@ -328,7 +328,7 @@ private long _getLongValue() throws JacksonException {
         try {
             if (getNumberType() == NumberType.BIG_INTEGER) {
                 BigInteger bigInteger = _reader.bigIntegerValue();
-                if (BI_MIN_INT.compareTo(bigInteger) > 0 || BI_MAX_INT.compareTo(bigInteger) < 0) {
+                if (BI_MIN_LONG.compareTo(bigInteger) > 0 || BI_MAX_LONG.compareTo(bigInteger) < 0) {
                     _reportOverflowLong();
                 }
                 return bigInteger.longValue();

ion/src/test/java/tools/jackson/dataformat/ion/IonNumberOverflowTest.java

@@ -5,6 +5,7 @@
 import org.hamcrest.Matchers;
 import org.junit.jupiter.api.Test;
 
+import tools.jackson.core.*;
 import tools.jackson.core.exc.InputCoercionException;
 
 import static org.hamcrest.MatcherAssert.assertThat;
@@ -72,4 +73,16 @@ private void _testLongCoercionFail(BigInteger input) throws Exception
         }
     }
 
+    // [dataformats-binary#569]: incorrect overflow fail for long values (from BigInteger)
+    @Test
+    public void testLongAsBigIntegerSize() throws Exception {
+        // Note: Values: Long.MAX_VALUE through Long.MAX_VALUE -7 are considered LONG by Ion.  
+        BigInteger bigIntLongValue = new BigInteger(Long.MAX_VALUE + "").subtract(BigInteger.TEN);
+        IonParser bigIntLongParser = (IonParser) new IonFactory().createParser(ObjectReadContext.empty(),
+                bigIntLongValue.toString());
+        assertEquals(JsonToken.VALUE_NUMBER_INT, bigIntLongParser.nextToken());
+        assertEquals(JsonParser.NumberType.BIG_INTEGER, bigIntLongParser.getNumberType());
+        assertEquals(JsonParser.NumberTypeFP.UNKNOWN, bigIntLongParser.getNumberTypeFP());
+        assertEquals(bigIntLongValue.longValue(), bigIntLongParser.getLongValue());
+    }    
 }

release-notes/VERSION-2.x

@@ -37,6 +37,9 @@ Active maintainers:
 
 2.18.4 (not yet released)
 
+#569: (ion) `IonParser` fails to parse some `long` values saying
+  they are out of range when they are not
+ (reported, fix suggested by @seadbrane)
 - (ion) Upgrade `ion-java` to 1.11.10 (from 1.11.9)
 
 2.18.3 (28-Feb-2025)