Title: `json.loads` ignores `sys.int_max_str_digits` (decodes an over-long int instead of raising `ValueError`) · Issue #8051 · RustPython/RustPython · GitHub
Open Graph Title: `json.loads` ignores `sys.int_max_str_digits` (decodes an over-long int instead of raising `ValueError`) · Issue #8051 · RustPython/RustPython
X Title: `json.loads` ignores `sys.int_max_str_digits` (decodes an over-long int instead of raising `ValueError`) · Issue #8051 · RustPython/RustPython
Description: Feature json.loads should honor the interpreter's integer string-conversion limit (sys.set_int_max_str_digits, PEP 651). RustPython's native _json scanner (the Rust-implemented accelerated module, RustPython's analog of CPython's C _json...
Open Graph Description: Feature json.loads should honor the interpreter's integer string-conversion limit (sys.set_int_max_str_digits, PEP 651). RustPython's native _json scanner (the Rust-implemented accelerated module, ...
X Description: Feature json.loads should honor the interpreter's integer string-conversion limit (sys.set_int_max_str_digits, PEP 651). RustPython's native _json scanner (the Rust-implemented accelerated ...
Opengraph URL: https://github.com/RustPython/RustPython/issues/8051
X: @github
Domain: github.com
{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"`json.loads` ignores `sys.int_max_str_digits` (decodes an over-long int instead of raising `ValueError`)","articleBody":"## Feature\n\n`json.loads` should honor the interpreter's integer string-conversion limit (`sys.set_int_max_str_digits`, PEP 651). RustPython's native `_json` scanner (the Rust-implemented accelerated module, RustPython's analog of CPython's C `_json`) decodes an over-long integer literal instead of raising `ValueError`. The pure-Python decoder already honors the limit (its int conversion goes through `int(str)`); only the native `_json` scanner diverges.\n\nSurfaced by running CPython's own stdlib test suite under RustPython — `test_json.test_decode::TestCDecode.test_limit_int` fails. The minimal standalone reproduction:\n\n### Reproduction\n\n```python\nimport sys, json\nsys.set_int_max_str_digits(5000)\njson.loads('1' * 5001)\n```\n\n**CPython 3.14:**\n\n```\nValueError: Exceeds the limit (5000 digits) for integer string conversion: value has 5001 digits\n```\n\n**RustPython:**\n\n```\n(returns the 5001-digit integer — no error)\n```\n\n## Python Documentation or reference to CPython source code\n\n- Limit / API: [`sys.set_int_max_str_digits`](https://docs.python.org/3/library/stdtypes.html#integer-string-conversion-length-limitation) (PEP 651).\n- CPython enforces it in [`_match_number_unicode`](https://github.com/python/cpython/blob/v3.14.3/Modules/_json.c), which builds the int via `PyLong_FromString` (limit-aware).\n- RustPython diverges in [`JsonScanner::parse_number`](https://github.com/RustPython/RustPython/blob/8e35cc9e72ea243df2705acb2cfb1a63fb8fbe0b/crates/stdlib/src/json.rs#L190), which builds the integer with `BigInt::from_str(buf)` and never consults `sys.int_max_str_digits`.\n\n### Suggested fix\n\nRoute the integer construction through `rustpython_common::int::bytes_to_int` with the live `vm.state.int_max_str_digits` (the same path the `int()` builtin uses), so the limit is read per-call and `sys.set_int_max_str_digits` / `adjust_int_max_str_digits` take effect. Float parsing is unaffected. This also unblocks `test_json.test_decode::TestCDecode.test_limit_int`.\n\n---\n*Investigated and drafted with AI assistance (Claude Code), reviewed by a human before filing, per RustPython's AI policy.*\n","author":{"url":"https://github.com/myers","@type":"Person","name":"myers"},"datePublished":"2026-06-05T16:17:50.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":0},"url":"https://github.com/8051/RustPython/issues/8051"}
| route-pattern | /_view_fragments/issues/show/:user_id/:repository/:id/issue_layout(.:format) |
| route-controller | voltron_issues_fragments |
| route-action | issue_layout |
| fetch-nonce | v2:7b94f073-a7ed-9776-494e-1698f7baf73d |
| current-catalog-service-hash | 81bb79d38c15960b92d99bca9288a9108c7a47b18f2423d0f6438c5b7bcd2114 |
| request-id | 9FF2:16F604:2B0A5E:3CC916:6A63689F |
| html-safe-nonce | 90bd822ba2a2bd0dbc4a071f3e19565fb551467310ca6f75457b5892921cf45a |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiI5RkYyOjE2RjYwNDoyQjBBNUU6M0NDOTE2OjZBNjM2ODlGIiwidmlzaXRvcl9pZCI6IjMyMTQ0NDY4NjQzNDA1NzY0MTUiLCJyZWdpb25fZWRnZSI6ImlhZCIsInJlZ2lvbl9yZW5kZXIiOiJpYWQifQ== |
| visitor-hmac | f77a11d5f359adebd2790a944571da18cbb7d8fb315e9e961c1bb7659178e184 |
| hovercard-subject-tag | issue:4598387242 |
| github-keyboard-shortcuts | repository,issues,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/_view_fragments/issues/show/RustPython/RustPython/8051/issue_layout |
| twitter:image | https://opengraph.githubassets.com/a6295fd4d9a69636a56c7ad0bf5535c093aa619a68c366737415e86077c68d3c/RustPython/RustPython/issues/8051 |
| twitter:card | summary_large_image |
| og:image | https://opengraph.githubassets.com/a6295fd4d9a69636a56c7ad0bf5535c093aa619a68c366737415e86077c68d3c/RustPython/RustPython/issues/8051 |
| og:image:alt | Feature json.loads should honor the interpreter's integer string-conversion limit (sys.set_int_max_str_digits, PEP 651). RustPython's native _json scanner (the Rust-implemented accelerated module, ... |
| og:image:width | 1200 |
| og:image:height | 600 |
| og:site_name | GitHub |
| og:type | object |
| og:author:username | myers |
| hostname | github.com |
| expected-hostname | github.com |
| None | 669d3fcd704a169521d2b6431762f95a6878a9e8ccaf0fd5f2d0802fc5b971f9 |
| turbo-cache-control | no-preview |
| go-import | github.com/RustPython/RustPython git https://github.com/RustPython/RustPython.git |
| octolytics-dimension-user_id | 39710557 |
| octolytics-dimension-user_login | RustPython |
| octolytics-dimension-repository_id | 135201145 |
| octolytics-dimension-repository_nwo | RustPython/RustPython |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 135201145 |
| octolytics-dimension-repository_network_root_nwo | RustPython/RustPython |
| turbo-body-classes | logged-out env-production page-responsive |
| disable-turbo | false |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 20da292f132ff51328be50073854259f417052c1 |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width