Relaxed constraint on clientID length #11

broker/src/main/java/org/eclipse/moquette/spi/impl/ProtocolProcessor.java

@@ -151,7 +151,7 @@ class ProtocolProcessor implements EventHandler<ValueEvent> {
             return;
         }
 
-        if (msg.getClientID() == null || msg.getClientID().length() > 23 || msg.getClientID().length() == 0) {
+        if (msg.getClientID() == null || msg.getClientID().length() == 0) {
             ConnAckMessage okResp = new ConnAckMessage();
             okResp.setReturnCode(ConnAckMessage.IDENTIFIER_REJECTED);
             session.write(okResp);

broker/src/test/java/org/eclipse/moquette/spi/impl/ProtocolProcessorTest.java

@@ -186,7 +186,7 @@ public class ProtocolProcessorTest {
         m_processor.processConnect(m_session, connMsg);
 
         //Verify
-        assertEquals(ConnAckMessage.IDENTIFIER_REJECTED, m_returnCode);
+        assertEquals(ConnAckMessage.CONNECTION_ACCEPTED, m_returnCode);
     }
 
     @Test