You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sling.apache.org by "Konrad Windszus (Jira)" <ji...@apache.org> on 2020/02/27 15:24:00 UTC

[jira] [Updated] (SLING-9163) Commons Johnzon is not fully JSON-P 1.1 compliant yet

     [ https://issues.apache.org/jira/browse/SLING-9163?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Konrad Windszus updated SLING-9163:
-----------------------------------
    Description: 
Commons Johnzon currently embeds Apache Johnzon 1.1.1 (https://github.com/apache/sling-org-apache-sling-commons-johnzon/blob/69a090477728ebabd23ff6b11dc2af8672229b9a/pom.xml#L72). Unfortunately that old version does not fully implement JSON-P 1.1 yet. E.g. support for JsonParser.getValue() has only been added in https://github.com/apache/johnzon/commit/05effff6e0a555d1a2c62e65d1891745df028ea7#diff-9f67203e03fd6aa15020bbf3bb822f8a which is part of 1.1.5+. Also some crucial fixes have been provided in https://github.com/apache/johnzon/commit/9871e297994a092d8e2492c58686bac6c23b56a4#diff-9f67203e03fd6aa15020bbf3bb822f8a (1.2.1+).

I consider this a bug, because the API for JSON-P 1.1 is already included and exported in version 1.1 but in fact {{Parser.getValue()}} will always throw a java.lang.UnsupportedOperationException although it is not supposed to according to file:///Users/konradwindszus/Downloads/javax.json-api-1.1-javadoc/javax/json/stream/JsonParser.html#getValue--

  was:
Commons Johnzon currently embeds Apache Johnzon 1.1.1 (https://github.com/apache/sling-org-apache-sling-commons-johnzon/blob/69a090477728ebabd23ff6b11dc2af8672229b9a/pom.xml#L72). Unfortunately that old version does not fully implement JSON-P 1.1 yet. E.g. support for JsonParser.getValue() has only been added in https://github.com/apache/johnzon/commit/05effff6e0a555d1a2c62e65d1891745df028ea7#diff-9f67203e03fd6aa15020bbf3bb822f8a which is part of 1.1.5+ and 1.2.3+. Also some crucial fixes have been provided in https://github.com/apache/johnzon/commit/9871e297994a092d8e2492c58686bac6c23b56a4#diff-9f67203e03fd6aa15020bbf3bb822f8a.

I consider this a bug, because the API for JSON-P 1.1 is already included and exported in version 1.1 but in fact {{Parser.getValue()}} will always throw a java.lang.UnsupportedOperationException although it is not supposed to according to file:///Users/konradwindszus/Downloads/javax.json-api-1.1-javadoc/javax/json/stream/JsonParser.html#getValue--


> Commons Johnzon is not fully JSON-P 1.1 compliant yet
> -----------------------------------------------------
>
>                 Key: SLING-9163
>                 URL: https://issues.apache.org/jira/browse/SLING-9163
>             Project: Sling
>          Issue Type: Bug
>    Affects Versions: Commons Johnzon 1.1.2
>            Reporter: Konrad Windszus
>            Assignee: Konrad Windszus
>            Priority: Major
>             Fix For: Commons Johnzon 1.1.4
>
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> Commons Johnzon currently embeds Apache Johnzon 1.1.1 (https://github.com/apache/sling-org-apache-sling-commons-johnzon/blob/69a090477728ebabd23ff6b11dc2af8672229b9a/pom.xml#L72). Unfortunately that old version does not fully implement JSON-P 1.1 yet. E.g. support for JsonParser.getValue() has only been added in https://github.com/apache/johnzon/commit/05effff6e0a555d1a2c62e65d1891745df028ea7#diff-9f67203e03fd6aa15020bbf3bb822f8a which is part of 1.1.5+. Also some crucial fixes have been provided in https://github.com/apache/johnzon/commit/9871e297994a092d8e2492c58686bac6c23b56a4#diff-9f67203e03fd6aa15020bbf3bb822f8a (1.2.1+).
> I consider this a bug, because the API for JSON-P 1.1 is already included and exported in version 1.1 but in fact {{Parser.getValue()}} will always throw a java.lang.UnsupportedOperationException although it is not supposed to according to file:///Users/konradwindszus/Downloads/javax.json-api-1.1-javadoc/javax/json/stream/JsonParser.html#getValue--



--
This message was sent by Atlassian Jira
(v8.3.4#803005)