jgallimore commented on a change in pull request #70:
URL: https://github.com/apache/johnzon/pull/70#discussion_r534094316
##########
File path:
johnzon-core/src/main/java/org/apache/johnzon/core/JsonProviderImpl.java
##########
@@ -53,6 +53,12 @@
import javax.json.stream.JsonParserFactory;
public class JsonProviderImpl extends JsonProvider implements Serializable {
+
+ /**
+ * This makes Johnzon to be compliant with spec, essentially regarding the
JSON Pointer /-
+ */
+ private final boolean isStrictCompliance =
Boolean.getBoolean("org.apache.johnzon.STRICT_JSONP_COMPLIANCE");
Review comment:
4. I did wonder the same, but then we potentially have a scenario where
the app server needs the setting to pass that TCK, and the setting isn't
available in when using Johnzon standalone - you then can't get the same
behaviour in both. Unit-testing is a use case where I could see this being
useful, for example.
While I'm sure we could solve this in the app server, I think my own
preference would be to enable the behaviour in Johnzon itself.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
