pay: ignore uncommited channels in listpeerchannels output

Uncommited channels are missing several fields which would normally be
populated by an active channel.  This simply skips them for the purposes
of finding a route.  The particular culprit was:
{
  "peer_id": "038cd9f3679d5b39bb2105978467918d549572de472f07dd729e37c7a6377d41d5",
  "peer_connected": true,
  "state": "OPENINGD",
  "owner": "lightning_openingd",
  "opener": "local",
  "to_us_msat": 8317559000,
  "total_msat": 8317559000,
  "features": [
    "option_static_remotekey",
    "option_anchors_zero_fee_htlc_tx"
  ]
}

Fixes #7197 - SEGV in direct_pay_listpeerchannels when field private missing

Changelog-Fixed: Fixed crash in pay plugin caused by parsing uncommitted dual open channels

plugins/libplugin.c

@@ -2115,8 +2115,14 @@ static struct listpeers_channel *json_to_listpeers_channel(const tal_t *ctx,
 
 	json_to_node_id(buffer, idtok, &chan->id);
 	json_to_bool(buffer, conntok, &chan->connected);
-	json_to_bool(buffer, privtok, &chan->private);
+	if (privtok)
+		json_to_bool(buffer, privtok, &chan->private);
+	else
+		chan->private = true;
 	chan->state = json_strdup(chan, buffer, statetok);
+	if (streq(chan->state, "DUALOPEND_OPEN_INIT") ||
+	    streq(chan->state, "OPENINGD"))
+		return tal_free(chan);
 	json_to_txid(buffer, ftxidtok, &chan->funding_txid);
 	if (scidtok != NULL) {
 		assert(dirtok != NULL);

tests/test_pay.py

@@ -5590,7 +5590,6 @@ def test_pay_partial_msat(node_factory, executor):
     l3pay.result(TIMEOUT)
 
 
-@pytest.mark.xfail()
 def test_pay_while_opening_channel(node_factory, bitcoind, executor):
     delay_plugin = {'plugin': os.path.join(os.getcwd(),
                                            'tests/plugins/openchannel_hook_delay.py'),