Github user michaelandrepearce commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/2490#discussion_r246128143 --- Diff: artemis-core-client/src/main/java/org/apache/activemq/artemis/core/protocol/core/impl/wireformat/SessionCreateConsumerMessage.java --- @@ -104,6 +119,11 @@ public void decodeRest(final ActiveMQBuffer buffer) { filterString = buffer.readNullableSimpleString(); browseOnly = buffer.readBoolean(); requiresResponse = buffer.readBoolean(); + if (buffer.readableBytes() > 0) { --- End diff -- This is typical pattern used for adding safely a new field that can be either nullable or defaultable. Used many times over.
---