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

Repair mixed close-brackets: using } instead of ] or the other way around #88

Closed
karam-khanna opened this issue Feb 17, 2023 · 4 comments
Labels
enhancement New feature or request

Comments

@karam-khanna
Copy link

Hello! Was hoping you could support JSON errors that looks like this:

{"topics": ["Hallstead Jewelers", "Lake Avenue and Second Avenue", "Washington Street", "Tiffany &" }}
or
{"topics": ["Hallstead Jewelers", "Lake Avenue and Second Avenue", "Washington Street", "Tiffany &" }]

@karam-khanna karam-khanna changed the title Support this kind of error Support this kind of incorrect JSON Feb 17, 2023
@josdejong josdejong changed the title Support this kind of incorrect JSON Repair mixed end-brackets: using } instead of ] or the other way around Feb 17, 2023
@josdejong josdejong added the enhancement New feature or request label Feb 17, 2023
@josdejong
Copy link
Owner

Good idea! so, basically repair an array that ends with } instead of ], and repair an object that ends with ] instead of }.

@josdejong josdejong changed the title Repair mixed end-brackets: using } instead of ] or the other way around Repair mixed close-brackets: using } instead of ] or the other way around Feb 17, 2023
@josdejong
Copy link
Owner

Thinking about it: the library already repairs missing closing brackets, so to address this, we should probably only remove redundant close brackets (which in itself is also good: repairing [1,2,3]] ).

@karam-khanna
Copy link
Author

karam-khanna commented Feb 17, 2023 via email

@josdejong
Copy link
Owner

This is supported now, fixed by implementing #103

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants