I have a patch for this and it was a trivial fix but Apache JIRA is currently down so will add it later when it comes back up.
If you want to make the fix locally look for the value(String value) method in JSWriter and replace the entire method body with out.print(outputQuotedString(x)); That should do the trick Rob On Jan 31, 2012, at 12:25 PM, Rurik Thomas Greenall wrote: > Rob — it's done :) > > On Tue, Jan 31, 2012 at 8:58 PM, Robert Vesse <[email protected]> wrote: > >> Ok I definitely see some unescaped data on line 533 >> >> Can you file a JIRA issue for this so it can be tracked formally, patching >> this should hopefully be relatively easy >> >> Rob >> >> On Jan 31, 2012, at 11:49 AM, Rurik Thomas Greenall wrote: >> >>> Here's the example: http://pastebin.com/3MWeF1Z3, it's RDF 1.0 data >> from a >>> Talis store. >>> >>> R >>> >>> On Tue, Jan 31, 2012 at 8:42 PM, Robert Vesse <[email protected] >>> wrote: >>> >>>> It uses the JSWriter class from the org.openjena.atlas package which >> looks >>>> like it does some escaping but may not necessarily escape everything it >>>> should. It looks like there is code for escaping but that the code path >>>> invoked does not necessarily call the escaping >>>> >>>> Do you have a specific example where invalid JSON is generated? >>>> >>>> Rob >>>> >>>> On Jan 31, 2012, at 11:33 AM, Rurik Thomas Greenall wrote: >>>> >>>>> Does JSONRDFWriter produce properly escaped JSON? I have noticed that >>>>> inverted commas are not escaped. >>>>> >>>>> Cheers, >>>>> >>>>> R. >>>> >>>> >> >>
