Stefan Seifert created SLING-6536:
-------------------------------------

             Summary: Replace JSON implementation derived from org.json code
                 Key: SLING-6536
                 URL: https://issues.apache.org/jira/browse/SLING-6536
             Project: Sling
          Issue Type: Improvement
          Components: Commons
    Affects Versions: Commons JSON 2.0.18
            Reporter: Stefan Seifert
            Priority: Critical
             Fix For: Commons JSON 2.1.0


following the discussion in
https://lists.apache.org/thread.html/ee51bace078681765d5dcfeda1939628ccefb9b4261b1d7f6a56d420@%3Cdev.sling.apache.org%3E
we have to replace the implementation of all classes in Commons JSON that were 
derived from the original org.json code.

the affected packages are
* 
[org.apache.sling.commons.json|https://github.com/apache/sling/tree/trunk/bundles/commons/json/src/main/java/org/apache/sling/commons/json]
* 
[org.apache.sling.commons.json.http|https://github.com/apache/sling/tree/trunk/bundles/commons/json/src/main/java/org/apache/sling/commons/json/http]
* 
[org.apache.sling.commons.json.io|https://github.com/apache/sling/tree/trunk/bundles/commons/json/src/main/java/org/apache/sling/commons/json/io]
* 
[org.apache.sling.commons.json.util|https://github.com/apache/sling/tree/trunk/bundles/commons/json/src/main/java/org/apache/sling/commons/json/util]
* 
[org.apache.sling.commons.json.xml|https://github.com/apache/sling/tree/trunk/bundles/commons/json/src/main/java/org/apache/sling/commons/json/xml]

unfortunately the unit test coverage for those packages is not very high. the 
main package has a line coverage of 42%, the other packages less than that or 
no at all.

although we might encourage modules to switch to a standard JSON implementation 
a lot of modules inside sling and perhaps much more outside sling depend on 
this json implementation, so we should try to keep the exported API and 
functionality.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to