I checked the impl and it seems safeWriteChar for escaping <>&" is only necessary for HTML output and not JSON one.
This PR fixes it by changing all usages of writeJSONString within JOSNRenderer to unescaped (my guess even for attrs it might be unnecessary but haven't touched it)
Also added small changes in test to cover regression + only make sure JSON renderer can pass the commonmark parsing. (TODO, still failing)
pi0
commented
6 months ago
Resolves #5
I checked the impl and it seems
safeWriteCharfor escaping<>&"is only necessary for HTML output and not JSON one.This PR fixes it by changing all usages of
writeJSONStringwithinJOSNRendererto unescaped (my guess even for attrs it might be unnecessary but haven't touched it)Also added small changes in test to cover regression +
only make sure JSON renderer can pass the commonmark parsing.(TODO, still failing)