fix for issue 1484 raised and solved by Graham Klyne: · Pull Request #1490 · RDFLib/rdflib (original) (raw)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FWIW, I was trying to home in on the error that was triggered by trying to parse the output from json.dump, which specifically caused the stream result from source.getByteStream() to be a character stream. I agree that seems not quite right, and the problem may lie deeper in the logic that sorts out the data stream to be parsed.

But I was also trying to identify a minimal fix that could solve my immediate problem. Hopefully if someone chooses to delve deeper into the create_input_source logic, the test cases provided will still be helpful.