Wait for ACK to remove stored message

Bringing in a change that's been on the production relay for a while but
apparently didn't pass tests locally. Now it does.
This commit is contained in:
Eric House 2019-03-17 11:52:48 -07:00
parent da917c35f5
commit 577fd3e560

View file

@ -1359,10 +1359,6 @@ handleMsgsMsg( const AddrInfo* addr, bool sendFull,
logf( XW_LOGVERBOSE0, "%s(): wrote %d bytes to socket %d", __func__,
nWritten, sock );
dbmgr->RecordSent( &msgIDs[0], msgIDs.size() );
// This is wrong: should be removed when ACK returns and not
// before. But for some reason if I make that change apps wind up
// stalling.
dbmgr->RemoveStoredMessages( msgIDs );
} else {
assert(0);
}