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

JsonLocation off for unrecognized tokens #1180

Open
cowtowncoder opened this issue Dec 31, 2023 · 0 comments
Open

JsonLocation off for unrecognized tokens #1180

cowtowncoder opened this issue Dec 31, 2023 · 0 comments
Labels
2.17 Issues planned (at earliest) for 2.17

Comments

@cowtowncoder
Copy link
Member

(note: follow up to remaining case from #1173)

Looks like JsonLocation reported for input like:

{"isThisValidJson": FALSE}

is off so that reported position of error is at the end of FALSE token, instead of the beginning (first character of unrecognized token).
@cowtowncoder cowtowncoder added the 2.17 Issues planned (at earliest) for 2.17 label Dec 31, 2023
cowtowncoder added a commit that referenced this issue Dec 31, 2023
@cowtowncoder
Cleave of failing #1180 tests from passing #1173 tests
717cd6b
@cowtowncoder cowtowncoder changed the title JsonLocation off for unreocgnized tokens JsonLocation off for unrecognized tokens Jan 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
2.17 Issues planned (at earliest) for 2.17
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@cowtowncoder