Subscription identifiers in subscribe is protocol error

The CONACK indicated that it's not supported, and the specs say to issue a disconnect when they're used after all.

Subscription identifiers in subscribe is protocol error
The CONACK indicated that it's not supported, and the specs say to issue a disconnect when they're used after all.
Wiebe Cazemier
1 parent e4b43df7
Showing 1 changed file with 1 additions and 2 deletions
mqttpacket.cpp
@@ -730,8 +730,7 @@ void MqttPacket::handleSubscribe()
             switch (prop)
             {
             case Mqtt5Properties::SubscriptionIdentifier:
-                decodeVariableByteIntAtPos();
-                break;
+                throw ProtocolError("Subscription identifiers not supported", ReasonCodes::ProtocolError);
             case Mqtt5Properties::UserProperty:
                 readUserProperty();
                 break;