vert-x3 / vertx-mqtt

Vert.x MQTT
Apache License 2.0
184 stars 86 forks source link

Adding the handler of vertx mqtt will cause the net read / write index of metrics to be unable to be obtained normally. Addlast should be used instead of addbefore #215

Open 1ssqq1lxr opened 2 years ago

1ssqq1lxr commented 2 years ago

Questions

Adding the handler of vertx mqtt will cause the net read / write index of metrics to be unable to be obtained normally. Addlast should be used instead of addbefore

Version

Which version(s) did you encounter this bug ?

Context

I encountered an exception which looks suspicious while ...

Do you have a reproducer?

A reproducer is a simple project hosted on GitHub (or another forge supporting git clone operation) that has a build file that can be executed to reproduce the issue.

Reproducers are very helpful for contributors and will likely help them fixing your bug faster.

Steps to reproduce

  1. ...
  2. ...
  3. ...
  4. ...

Extra

1ssqq1lxr commented 2 years ago

VertxHandler : protected void reportBytesRead(Object msg) { if (msg instanceof ByteBuf) { this.reportBytesRead((long)((ByteBuf)msg).readableBytes()); }

}

MqttEncoder && MqttDecoder is before VertxHandler ,so msg is Mqtt object