splice: More agressive checks for tx_abort

In interop with Eclair t-bast believes we’re ignoring `tx_abort` during reestablish. Going through and auditing all the reestablish flows, improving some logs around it and adding checks for `tx_abort` coming in before `channel_reestablish`.

ChangelogNone

Author: ddustin

channeld/channeld.c

@@ -3019,9 +3019,11 @@ static const u8 *peer_expect_msg_three(const tal_t *ctx,
 	    && type != third_allowed_type)
 		peer_failed_warn(peer->pps, &peer->channel_id,
 				"Got incorrect message from peer: %s"
-				" (should be %s) [%s]",
+				" (should be %s or %s or %s) [%s]",
 				peer_wire_name(type),
 				peer_wire_name(expect_type),
+				peer_wire_name(second_allowed_type),
+				peer_wire_name(third_allowed_type),
 				sanitize_error(tmpctx, msg, &peer->channel_id));
 
 	return msg;
@@ -5705,6 +5707,8 @@ static void peer_reconnect(struct peer *peer,
 	do {
 		clean_tmpctx();
 		msg = peer_read(tmpctx, peer->pps);
+		check_tx_abort(peer, msg,
+			       inflight ? &inflight->outpoint.txid : NULL);
 	} while (handle_peer_error_or_warning(peer->pps, msg) ||
 		 capture_premature_msg(&premature_msgs, msg));