ACKs not sent for retried messages

[rxt1077]

When using reliable routing (wantAck) the first packet is acknowledged, but if retries are sent they are not acknowledged. This means a single missed ACK will cause a reliable send to fail. This is in contrast to the expected behavior, stated in this comment, which is to resend an ACK for each repeated packet.

This occurs because ReliableRouter::shouldFilterRecieved calls FloodingRouter::shouldFilterRecieved which filters the packet before ReliableRouter::sniffRecieved (which handles most of the ACK logic) even sees it.

[geeksville]

This issue has been mentioned on Meshtastic. There might be relevant details there:

https://meshtastic.discourse.group/t/meshtastic-device-1-2-48-alpha/4359/1