Github user jbertram commented on a diff in the pull request:
https://github.com/apache/activemq-artemis/pull/1592#discussion_r145437240
--- Diff:
artemis-protocols/artemis-mqtt-protocol/src/main/java/org/apache/activemq/artemis/core/protocol/mqtt/MQTTUtil.java
---
@@ -67,6 +76,8 @@
public static final int DEFAULT_KEEP_ALIVE_FREQUENCY = 5000;
+ public static final String logUTF8PublishPayload =
System.getProperty("org.apache.activemq.artemis.core.protocol.mqtt.logUTF8PublishPayload");
--- End diff --
To keep this as simple as possible I'll get rid of the system property and
always log the payload as UTF-8, but also always limit the output to a certain
number of characters so we don't get hundreds of megs of data (potentially).
---