Skip to content

Commit c6bfda6

Browse files
Add todo comment about future TCP ACK retransmit testing
1 parent ac8474d commit c6bfda6

File tree

1 file changed

+6
-0
lines changed

1 file changed

+6
-0
lines changed

src/EthernetClient.cpp

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -92,6 +92,12 @@ int EthernetClient::available()
9292
{
9393
if (sockindex >= MAX_SOCK_NUM) return 0;
9494
return Ethernet.socketRecvAvailable(sockindex);
95+
// TODO: do the Wiznet chips automatically retransmit TCP ACK
96+
// packets if they are lost by the network? Someday this should
97+
// be checked by a man-in-the-middle test which discards certain
98+
// packets. If ACKs aren't resent, we would need to check for
99+
// returning 0 here and after a timeout do another Sock_RECV
100+
// command to cause the Wiznet chip to resend the ACK packet.
95101
}
96102

97103
int EthernetClient::read(uint8_t *buf, size_t size)

0 commit comments

Comments
 (0)