René's URL Explorer Experiment

{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"Jackson 3: numeric functions/operators throw on out-of-range values ($fromMillis, $sum, unary minus, range, array index)","articleBody":"## Summary\n\nAfter the Jackson 3 migration (#430, PR #431), several numeric functions and operators call `asInt()`/`asLong()` on values that are outside `int`/`long` range. Jackson 3 makes these coercions **fail‑fast** — they throw `tools.jackson.databind.node.JsonNodeException` for out‑of‑range `DoubleNode`/`BigIntegerNode` values. Jackson 2 silently narrowed (a plain cast), so these paths previously produced a (possibly wrong but non‑throwing) value or a controlled JSONata error. The lines were not changed by the migration, so behavior flipped from \"return/handle\" to \"crash\".\n\nNot caught by the test suite — no test exercises out‑of‑range magnitudes at these sites.\n\n## Where \u0026 reproductions\n\n| Site | Code | Repro | Jackson 2 | Jackson 3 |\n| --- | --- | --- | --- | --- |\n| `ExpressionsVisitor.java:381` `isWholeNumber` | `n.asInt() == n.asDouble()` | `[1..1e19]` (range bound) | returns `false` → controlled \"must be an integer\" error | **throws** raw `JsonNodeException` |\n| `ExpressionsVisitor.java:994` (array index) | `indexInContext.asInt()` | `[\"a\",\"b\"][3000000000]` | narrows → harmless out‑of‑bounds | **throws** |\n| `ExpressionsVisitor.java:2919/2925` (unary minus) | `operand.asLong()` on `BigIntegerNode` | `-99999999999999999999999999` | narrowed long | **throws** |\n| `FromMillisFunction.java:90` | `argNumber.asLong()` (guarded only by `isNumber()`) | `$fromMillis(1e30)` | narrowed → wrong date | **throws** |\n| `FromMillisZonedFunction.java:90` | `argNumber.asLong()` | `$fromMillisTz(1e30, '[Y]')` | narrowed | **throws** |\n| `SumFunction.java:95` and `:107` | `a.asLong()` / `argArray.asLong()` on out‑of‑range `BigIntegerNode` | `$sum([99999999999999999999999999])` | narrowed | **throws** |\n\n## Verified empirically against jackson‑databind 3.2.0\n\n```\nDoubleNode(1.0E46).asInt()   -\u003e THROWS JsonNodeException\nDoubleNode(1.0E46).asLong()  -\u003e THROWS JsonNodeException\nBigIntegerNode(99999999999999999999).asLong() -\u003e THROWS JsonNodeException\nLongNode(3000000000).asInt() -\u003e THROWS JsonNodeException\n```\n\n## Suggested fix\n\nBefore calling `asInt()`/`asLong()` at these sites, pre‑check magnitude (`canConvertToInt()` / `canConvertToLong()`) or operate on `doubleValue()`/`bigIntegerValue()` and range‑check, restoring the Jackson‑2 narrowing (or converting to the appropriate controlled JSONata error) rather than allowing a raw `JsonNodeException` to escape. This mirrors the guard already applied to `AgnosticTestSuite`'s `(long) d` fix in this migration.\n","author":{"url":"https://github.com/wnm3","@type":"Person","name":"wnm3"},"datePublished":"2026-07-07T16:39:05.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":1},"url":"https://github.com/433/JSONata4Java/issues/433"}