mcollina / aedes / 145

98%

master: 98%

push

travis-ci

<a href="https://github.com/mcollina/aedes/commit/<a class=hub.com/mcollina/aedes/commit/dc650dcdfe603ece773994ec4640e2152a5793e7">dc650dcdf<a href="https://github.com/mcollina/aedes/commit/dc650dcdfe603ece773994ec4640e2152a5793e7">&quot;&gt;Enhance &amp;amp; Optimize connect handler
- For https://github.com/mcollina/aedes/pull/260/commits/</a><a class="double-link" href="https://github.com/mcollina/aedes/commit/<a class="double-link" href="https://github.com/mcollina/aedes/commit/06fc8392a1ffea822d003ff6f4c01d100b0e616d">06fc8392a</a>">06fc8392a</a><a href="https://github.com/mcollina/aedes/commit/dc650dcdfe603ece773994ec4640e2152a5793e7">, raise an &quot;invalid protocol&quot; callback error if there is

- aedes unsupported mqtt version: CONNACK return Code = 1
- client id length &gt; 23 [MQTT 3.1.0 only]: CONNACK return Code = 2

- Added preConnect handler in handleConnect between earliest connect checks and normal checks. This is useful for users if they want to do some earilest DDoS check before server send any responses back, in this phrase connected=false

- Emit `connackSent` event and set connactSent=true if CONNACK is sent, not only after `client` event but also when normal connect checks phrase if necessary

- Added packet arguments in `connackSent` event

- set clientID to &#39;aedes_&#39; + shortid() if empty [MQTT 3.1.1], it is better to keep it within 23 chars for better compatibility

- Emit `clientReady` event after we send back all offline messages to client

- Optimize negate function

- Optimize doConnack function and we could re-use it

- Set keepalive after authentication, save some resoures if there are plenty of failed authentication

264 of 285 branches covered (92.63%)

695 of 711 relevant lines covered (97.75%)

6911.04 hits per line