`UTF8DataInputJsonParser` reports end-of-input as `JacksonIOException` instead of `UnexpectedEndOfInputException`
Maintainers usually reply within 1 day
Assessment
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Newbie friendliness
- 57/100
Research direction
Start by comparing EOF handling in UTF8DataInputJsonParser and UTF8StreamJsonParser, then trace how public entry points such as nextToken() and nextName() report errors with the current context or token. The issue suggests central mapping of EOFException to _reportInvalidEOF(); check existing parser tests for the listed inputs and add coverage for matching exception types and useful messages.
Written by the indexing model from the issue text.
Description
UTF8DataInputJsonParser reports most premature end-of-input as JacksonIOException (wrapping EOFException), whereas UTF8StreamJsonParser throws UnexpectedEndOfInputException for the same content.
| Input | InputStream | DataInput |
|---|---|---|
"abc |
UnexpectedEndOfInputException (in VALUE_STRING) |
JacksonIOException |
{ "abc |
UnexpectedEndOfInputException (in property name) |
JacksonIOException |
{"a": |
UnexpectedEndOfInputException (within/between Object entries) |
JacksonIOException |
[ 12 |
UnexpectedEndOfInputException (expected close marker) |
JacksonIOException |
+ |
UnexpectedEndOfInputException (in a Number value) |
JacksonIOException |
With java.io.DataInputStream the message is just "N/A", since its EOFException has no message.
Only a few sites handle EOFException explicitly (_skipWSOrEnd(), escape decoding, and invalid-token reporting since #1752). Instead of adding more per-site catches, EOFException could be mapped to _reportInvalidEOF() centrally in nextToken(), nextName() and other public entry points, using the current context/token for the message.
Related: #325 (root-level end-of-input), #1752.
- Dominant language
- Java
- Stars
- 2.4k
- Forks
- 952
- Avg merge
- 3d 6h
- Merged PRs (30d)
- 32
Getting set up
This project ships no dev container, Dockerfile or contributing guide, so setting up is up to you: start from its README, and see our first-contribution guide for the general steps.
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from FasterXML/jackson-core
-
Concurrent `BufferRecycler.releaseToPool()` calls can return the same recycler more than oncePossibly taken @Dongnyoung claimed this 5 days ago. Open
Difficulty 3/5 1-2 days Newbie friendliness 70/100
FasterXML/jackson-core#1824 ·
Maintainers usually reply within 1 day
-
3.4
Difficulty 5/5 Over a week Newbie friendliness 38/100
FasterXML/jackson-core#1740 · 5 comments · 1 reaction ·
Maintainers usually reply within 1 day
-
`maxStringLength` is not enforced when skipping String valuesPossibly taken @Dongnyoung claimed this 39 days ago. Open
Difficulty 3/5 1-2 days Newbie friendliness 75/100
FasterXML/jackson-core#1735 · 2 comments · 1 assignee ·
Maintainers usually reply within 1 day
-
Consider validating raw unpaired UTF-16 surrogates in ReaderBasedJsonParserPossibly taken @nvsaigokul-sudo claimed this 7 days ago. Open
Difficulty 5/5 Over a week Newbie friendliness 35/100
FasterXML/jackson-core#1705 · 5 comments · 1 reaction ·
Maintainers usually reply within 1 day
-
Non-root number separator validation not applied to non-blocking (async) parser (follow-up to #1557)Possibly taken @seonwooj0810 claimed this 98 days ago. Open
Difficulty 4/5 3-5 days Newbie friendliness 68/100
FasterXML/jackson-core#1640 ·
Maintainers usually reply within 1 day
All issues in FasterXML/jackson-core
Similar issues
-
Cleanup: restore the interrupt flag in AiFailureAnalyzerPossibly taken @Rishankgupta08 claimed this today. Opengood first issue
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
seleniumboot/selenium-boot#258 · 1 comment ·
Maintainers usually reply within 1 day
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
dense-analysis/ale#5181 ·
-
bug subsystem:agent-harness triage
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
ShaftHQ/SHAFT_ENGINE#8043 ·
Maintainers usually reply within 1 day
-
Difficulty 1/5 Under an hour Newbie friendliness 88/100
johanhaleby/occurrent#1505 ·
Maintainers usually reply within 1 day
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
Maintainers usually reply within 1 day