Github user michaelandrepearce commented on a diff in the pull request:

    https://github.com/apache/activemq-artemis/pull/2238#discussion_r209438598
  
    --- Diff: 
artemis-jms-server/src/main/java/org/apache/activemq/artemis/jms/server/config/impl/ConnectionFactoryConfigurationImpl.java
 ---
    @@ -623,6 +636,8 @@ public void decode(final ActiveMQBuffer buffer) {
           deserializationBlackList = 
BufferHelper.readNullableSimpleStringAsString(buffer);
     
           deserializationWhiteList = 
BufferHelper.readNullableSimpleStringAsString(buffer);
    +
    +      enable1xPrefixes = buffer.readBoolean();
    --- End diff --
    
    should check buffer has readable bytes before reading, for compatibility if 
the buffer is written by a previous version


---

Reply via email to