java.lang.IllegalArgumentException: Duplicate handler name: idle In 3.5.1

[dushibaiyu]

When set NetServer Option IdleTimeout > 0
In file io.vertx.core.net.impl.NetServerImpl at function initChannel at line 145~147 will add 'idle' handler to pipeline

And in file io.vertx.mqtt.impl.MqttServerImpl on function initChannel at line 141 will add a 'idle' hander too, it will throw the erro exception

I think should change the last 'idle' to 'mqttIdle'

[Sammers21]

@dushibaiyu, can you try with the 3.6.0-SNAPSHOT version?

[dushibaiyu]

@Sammers21 in the master it will has the error:
Beause in io.vertx.mqtt.impl.MqttServerImpl on function initChannel at line 141: pipeline.addBefore("handler", "idle", new IdleStateHandler(this.options.timeoutOnConnect(), 0, 0));

the "idle" handle is add in NetServerImpl before when you set NetServerOption.IdleTimeout > 0

[Sammers21]

Ah, I see. You are right. We have already fixed the issue in the MQTT client. See the PR: #79.

I think we can do the same thing for MQTT server.

@dushibaiyu, are you interested in contribution?

[dushibaiyu]

I have created a Pull-request.