Apply PR comments

project-chip · Nov 11, 2021 · 5c4125b · 5c4125b
1 parent 10e1c88
commit 5c4125b
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/src/messaging/ExchangeContext.cpp b/src/messaging/ExchangeContext.cpp
@@ -139,8 +139,9 @@ CHIP_ERROR ExchangeContext::SendMessage(Protocols::Id protocolId, uint8_t msgTyp
     // Treat unknown peer address as "not UDP", because we have no idea whether
     // it's safe to do MRP there.
     bool isUDPTransport = peerAddress && peerAddress->GetTransportType() == Transport::Type::kUdp;
-    bool reliableTransmissionRequested =
-        isUDPTransport && !sendFlags.Has(SendMessageFlags::kNoAutoRequestAck) && !IsGroupExchangeContext();
+
+    // this check is ignored by the ExchangeMsgDispatch if !AutoRequestAck()
+    bool reliableTransmissionRequested = isUDPTransport && !sendFlags.Has(SendMessageFlags::kNoAutoRequestAck);
 
     // If a response message is expected...
     if (sendFlags.Has(SendMessageFlags::kExpectResponse) && !IsGroupExchangeContext())