Sanitize JSON-LD input before decoding #41
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR relies on #38 getting merged, as commit e4e9a18 is the commit that fixes the bug
When the Json ends with a
;
the json can not get decoded.Example
WHO content ends with an
;
. As result of that theNewsArticle
can not get parsed.https://www.who.int/news-room/detail/17-10-2019-7-million-people-receive-record-levels-of-lifesaving-tb-treatment-but-3-million-still-miss-out
How to test
Run while parsing above URL
Or test the following JSON snippet on https://jsonlint.com/#:
How to fix
check if the last character is a
;
, and strip that