[jira] [Commented] (ARTEMIS-3607) JsonUtil addToObject should be able to add a JsonValue

["ASF subversion and git services (Jira)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/ARTEMIS-3607?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17459946#comment-17459946 ] 

ASF subversion and git services commented on ARTEMIS-3607:
----------------------------------------------------------

Commit dd645d0d4ef7526804043bdbb3850546eba16d76 in activemq-artemis's branch refs/heads/main from Emmanuel Hugonnet
[ https://gitbox.apache.org/repos/asf?p=activemq-artemis.git;h=dd645d0 ]

[ARTEMIS-3607]: Supporting JsonValues in JsonUtil.addToArray and
JsonUtil.addToObject

* When the added Object is of type JsonValue, don't call the toString
  method on it.

Issue: https://issues.apache.org/jira/browse/ARTEMIS-3607


> JsonUtil addToObject should be able to add a JsonValue
> ------------------------------------------------------
>
>                 Key: ARTEMIS-3607
>                 URL: https://issues.apache.org/jira/browse/ARTEMIS-3607
>             Project: ActiveMQ Artemis
>          Issue Type: Bug
>          Components: Broker
>    Affects Versions: 2.19.0
>            Reporter: Emmanuel Hugonnet
>            Priority: Major
>          Time Spent: 20m
>  Remaining Estimate: 0h
>
> Adding a JsonObject to a JsonObject using JsonUtil.addToObject will have a NPE if the value is JsonValue.NULL (NullableJsonString). When adding a JsonObject (aka a Map of JsonValues) we should use the to String method but 'just' add the JsonValue



--
This message was sent by Atlassian Jira
(v8.20.1#820001)