lightningd: notify plugins when finalizing channel

Changelog-Added: Plugins now receive `channel_state_changed` notification upon final change to `CLOSED` state.

Author: whitslack

lightningd/channel.c

@@ -87,7 +87,8 @@ static void destroy_channel(struct channel *channel)
 	list_del_from(&channel->peer->channels, &channel->list);
 }
 
-void delete_channel(struct channel *channel STEALS, bool completely_eliminate)
+void delete_channel(struct channel *channel STEALS,
+		    bool completely_eliminate)
 {
 	const u8 *msg;
 	struct peer *peer = channel->peer;
@@ -111,6 +112,16 @@ void delete_channel(struct channel *channel STEALS, bool completely_eliminate)
 			fatal("HSM gave bad hsm_forget_channel_reply %s", tal_hex(msg, msg));
 	}
 
+	notify_channel_state_changed(channel->peer->ld,
+				     &channel->peer->id,
+				     &channel->cid,
+				     channel->scid,
+				     time_now(),
+				     channel->state,
+				     CLOSED,
+				     REASON_UNKNOWN,
+				     NULL);
+
 	tal_free(channel);
 
 	maybe_delete_peer(peer);

lightningd/channel.h

@@ -493,7 +493,8 @@ channel_current_inflight(const struct channel *channel);
 u32 channel_last_funding_feerate(const struct channel *channel);
 
 /* Only set completely_eliminate for never-existed channels */
-void delete_channel(struct channel *channel STEALS, bool completely_eliminate);
+void delete_channel(struct channel *channel STEALS,
+		    bool completely_eliminate);
 
 /* Add a historic (public) short_channel_id to this channel */
 void channel_add_old_scid(struct channel *channel,

wallet/test/run-wallet.c

@@ -758,17 +758,6 @@ void notify_channel_mvt(struct lightningd *ld UNNEEDED,
 void notify_channel_open_failed(struct lightningd *ld UNNEEDED,
                                 const struct channel_id *cid UNNEEDED)
 { fprintf(stderr, "notify_channel_open_failed called!\n"); abort(); }
-/* Generated stub for notify_channel_state_changed */
-void notify_channel_state_changed(struct lightningd *ld UNNEEDED,
-				  const struct node_id *peer_id UNNEEDED,
-				  const struct channel_id *cid UNNEEDED,
-				  const struct short_channel_id *scid UNNEEDED,
-				  struct timeabs timestamp UNNEEDED,
-				  enum channel_state old_state UNNEEDED,
-				  enum channel_state new_state UNNEEDED,
-				  enum state_change cause UNNEEDED,
-				  const char *message UNNEEDED)
-{ fprintf(stderr, "notify_channel_state_changed called!\n"); abort(); }
 /* Generated stub for notify_connect */
 void notify_connect(struct lightningd *ld UNNEEDED,
 		    const struct node_id *nodeid UNNEEDED,
@@ -1313,6 +1302,17 @@ u8 *wire_sync_read(const tal_t *ctx UNNEEDED, int fd UNNEEDED)
 void plugin_hook_db_sync(struct db *db UNNEEDED)
 {
 }
+void notify_channel_state_changed(struct lightningd *ld UNNEEDED,
+				  const struct node_id *peer_id UNNEEDED,
+				  const struct channel_id *cid UNNEEDED,
+				  const struct short_channel_id *scid UNNEEDED,
+				  struct timeabs timestamp UNNEEDED,
+				  enum channel_state old_state UNNEEDED,
+				  enum channel_state new_state UNNEEDED,
+				  enum state_change cause UNNEEDED,
+				  const char *message UNNEEDED)
+{
+}
 bool fromwire_hsmd_get_channel_basepoints_reply(const void *p UNNEEDED,
 					       struct basepoints *basepoints,
 					       struct pubkey *funding_pubkey)