Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

JsonNode.requiredAt() does NOT fail on some path expressions #3005

Closed
cowtowncoder opened this issue Jan 2, 2021 · 0 comments
Closed

JsonNode.requiredAt() does NOT fail on some path expressions #3005

cowtowncoder opened this issue Jan 2, 2021 · 0 comments
Milestone
2.12.1

Comments

@cowtowncoder
Copy link
Member

Looks like JsonNode.requiredAt() does not catch one type of missing references: those trying to traverse through non-container (scalar) nodes:

For example, for document:

{ "simple" : 5 }

should fail for

JsonNode n = mapper.readTree("{\"simple\":5}");
n.requiredAt("/simple/property")

but it does not.
@cowtowncoder cowtowncoder added to-evaluate Issue that has been received but not yet evaluated 2.12 and removed to-evaluate Issue that has been received but not yet evaluated labels Jan 2, 2021
@cowtowncoder cowtowncoder modified the milestones: 1.9.13, 2.12.1 Jan 2, 2021
cowtowncoder added a commit that referenced this issue Jan 3, 2021
@cowtowncoder
... and fixing reference, fixed #3005, not #3003
972db89
cowtowncoder added a commit that referenced this issue Feb 20, 2021
@cowtowncoder
Fix #3005
b2e92b6
cowtowncoder added a commit that referenced this issue Feb 20, 2021
@cowtowncoder
Fix ref to issue #3008 (NOT #3005)
0df9053
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
2.12.1
Development

No branches or pull requests
1 participant
@cowtowncoder