fix for issue 1484 raised and solved by Graham Klyne:

rdflib/plugins/shared/jsonld/util.py

@@ -17,26 +17,33 @@
 
 from urllib.parse import urljoin, urlsplit, urlunsplit
 
-from rdflib.parser import create_input_source, PythonInputSource
+from rdflib.parser import create_input_source, PythonInputSource, StringInputSource
 
-from io import StringIO
+from io import TextIOBase, TextIOWrapper
 
 
 def source_to_json(source):
 
     if isinstance(source, PythonInputSource):
         return source.data
 
+    if isinstance(source, StringInputSource):
+        return json.load(source.getCharacterStream())
+
     # TODO: conneg for JSON (fix support in rdflib's URLInputSource!)
     source = create_input_source(source, format="json-ld")
 
     stream = source.getByteStream()
     try:
-        return json.load(StringIO(stream.read().decode("utf-8")))
+        # Use character stream as-is, or interpret byte stream as UTF-8
+        if isinstance(stream, TextIOBase):
+            use_stream = stream
+        else:
+            use_stream = TextIOWrapper(stream, encoding='utf-8')
+        return json.load(use_stream)
     finally:
         stream.close()
 
-
 VOCAB_DELIMS = ("#", "/", ":")
 
 

test/test_issue1484.py

@@ -0,0 +1,65 @@
+import unittest
+import io
+import json
+from rdflib import Graph, RDF, RDFS, Namespace
+
+
+class TestIssue1484_json(unittest.TestCase):
+    def test_issue_1484_json(self):
+        """
+        Test JSON-LD parsing of result from json.dump
+        """
+        n = Namespace("http://example.org/")
+        jsondata = {"@id": n.s, "@type": [n.t], n.p: {"@id": n.o}}
+
+        s = io.StringIO()
+        json.dump(jsondata, s, indent=2, separators=(",", ": "))
+        s.seek(0)
+
+        DEBUG = False
+        if DEBUG:
+            print("S: ", s.read())
+            s.seek(0)
+
+        b = n.base
+        g = Graph()
+        g.bind("rdf", RDF)
+        g.bind("rdfs", RDFS)
+        g.parse(source=s, publicID=b, format="json-ld")
+
+        assert (n.s, RDF.type, n.t) in g
+        assert (n.s, n.p, n.o) in g
+
+
+class TestIssue1484_str(unittest.TestCase):
+    def test_issue_1484_str(self):
+        """
+        Test JSON-LD parsing of result from string (used by round tripping tests)
+
+        (Previously passes, but broken by earlier fix for above.)
+        """
+        n = Namespace("http://example.org/")
+        jsonstr = """
+            {
+              "@id": "http://example.org/s",
+              "@type": [
+                "http://example.org/t"
+              ],
+              "http://example.org/p": {
+                "@id": "http://example.org/o"
+              }
+            }
+        """
+
+        b = n.base
+        g = Graph()
+        g.bind("rdf", RDF)
+        g.bind("rdfs", RDFS)
+        g.parse(data=jsonstr, publicID=b, format="json-ld")
+
+        assert((n.s, RDF.type, n.t) in g)
+        assert((n.s, n.p, n.o) in g)
+
+
+if __name__ == "__main__":
+    unittest.main()