Rob,
Thanks for this — your work is appreciated!
R
On 31. jan. 2012, at 22.10, Robert Vesse <[email protected]> wrote:
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.
