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.

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

Improve detection for some invalid JSON #146

Closed
wants to merge 1 commit into from
Closed

Improve detection for some invalid JSON #146

wants to merge 1 commit into from

Conversation

Crayon-new
Copy link

I found that some invalid JSON will be parsed as valid when they are in some situations. Also mentioned in #127

  • the beginning character is """, but it should be "[" or "{" .
  • the beginning character is "[" , but the end char is not "]".
  • the beginning character is "[" , but the end char is not "]".
    The json lib will parsed the input string only check the first char. So this invalid json will be parsed. So i also check the end of the json. And when there is ":" in the string. It shouldn't be seen as a JSONString.

@Crayon-new Crayon-new closed this Jan 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant