Github user michaelandrepearce commented on a diff in the pull request:
https://github.com/apache/activemq-artemis/pull/2238#discussion_r209819327
--- 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.readableBytes() > 0 ? buffer.readBoolean()
: null;
--- End diff --
@clebertsuconic @jbertram Should default to 0, null would throw npe here,
as field being set is a promiti5
---
