Skip to content

Commit

Permalink
chaintopology: add REORG to logline when tx watch is fired after reorg
Browse files Browse the repository at this point in the history
This makes clear that txwatch_fire was called with depth=0 when
chain tip got removed after a reorg. And now we can test for that.
  • Loading branch information
SimonVrouwe authored and cdecker committed May 27, 2019
1 parent db57d9c commit b261c5b
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions lightningd/watch.c
Original file line number Diff line number Diff line change
Expand Up @@ -220,10 +220,12 @@ static bool txw_fire(struct txwatch *txw,
else
log = txw->topo->log;

/* We assume zero depth signals a reorganization */
log_debug(log,
"Got depth change %u->%u for %s",
"Got depth change %u->%u for %s%s",
txw->depth, depth,
type_to_string(tmpctx, struct bitcoin_txid, &txw->txid));
type_to_string(tmpctx, struct bitcoin_txid, &txw->txid),
depth ? "" : " REORG");
txw->depth = depth;
r = txw->cb(txw->topo->bitcoind->ld, txw->channel, txid, txw->depth);
switch (r) {
Expand Down

0 comments on commit b261c5b

Please sign in to comment.