upczsh commented on code in PR #828:
URL: https://github.com/apache/knox/pull/828#discussion_r1440024716
##########
gateway-provider-rewrite/src/main/java/org/apache/knox/gateway/filter/rewrite/impl/json/JsonFilterReader.java:
##########
@@ -77,6 +77,11 @@ class JsonFilterReader extends Reader {
bufferingLevel = null;
bufferingConfig = null;
this.config = config;
+ jsonParserConfigInit();
+ }
+
+ private void jsonParserConfigInit() {
+
parser.enable(com.fasterxml.jackson.core.JsonParser.Feature.ALLOW_NON_NUMERIC_NUMBERS);
Review Comment:
@smolnar82 Thanks for your guidance, I have used
JsonReadFeature.ALLOW_NON_NUMERIC_NUMBERS instead of the deprecated
JsonParser.Feature.ALLOW_NON_NUMERIC_NUMBERS.
On the other hand, the reason for not adding it on line 69 is that this
setting is the content of parse, and other json parse configurations may be
enabled in the future, so I wrote a separate method.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
