Subscribing to MqttServer.ClientAcknowledgedPublishPacketAsync results in two events for every published application message

[mark-trumed]

Describe the bug

After creating an MqttServer instance, I add an event handler for ClientAcknowledgedPublishPacketAsync:
_mqttServer.ClientAcknowledgedPublishPacketAsync += OnClientAcknowledgedPublishPacketAsync;

Every time a message is published to a client and acknowledged by the client, the event handler function OnClientAcknowledgedPublishPacketAsync() is called twice.

Which component is your bug related to?

• Server

To Reproduce

Using version 4.1.2.350 of MQTTnet.
See the description for steps to reproduce.

Expected behavior

When a client receives and acknowledges a message, the ClientAcknowledgedPublishPacketAsync event handler is only called once.

Screenshots

n/a

Additional context / logging

n/a

[chkr1011]

Do you probably publish as QoS 2? It seems that the server then fires the event for the PUBREC and PUBCOMP packets.

[mark-trumed]

@chkr1011 Yes, messages were published as QoS 2.

[chkr1011]

Ok then I assume this is the issue. I will fix this so that the event only gets called after the PUBCOMP message is received because only this message clearly marks the successful end of the transfer process.

[mark-trumed]

@chkr1011 Thank you.