openingd: make channel-type rejection message clearer.

For example, lnprototest got the error 'You gave bad parameters: Did not support channel_type ' which doesn't make it clear that it's rejecting the empty channel type.

Signed-off-by: Rusty Russell <[email protected]>

Author: rustyrussell

openingd/dualopend.c

@@ -2433,7 +2433,7 @@ static void accepter_start(struct state *state, const u8 *oc2_msg)
 				state->channel_type = channel_type_from(state, open_tlv->channel_type);
 			} else {
 				negotiation_failed(state,
-						   "Did not support channel_type %s",
+						   "Did not support channel_type [%s]",
 						   fmt_featurebits(tmpctx,
 								   open_tlv->channel_type));
 				return;

openingd/openingd.c

@@ -974,7 +974,7 @@ static u8 *fundee_channel(struct state *state, const u8 *open_channel_msg)
 				state->channel_type = channel_type_from(state, open_tlvs->channel_type);
 			} else {
 				negotiation_failed(state,
-						   "Did not support channel_type %s",
+						   "Did not support channel_type [%s]",
 						   fmt_featurebits(tmpctx,
 								   open_tlvs->channel_type));
 				return NULL;

tests/test_opening.py

@@ -2654,7 +2654,7 @@ def test_opening_explicit_channel_type(node_factory, bitcoind):
     l1.start()
     l1.connect(l2)
 
-    with pytest.raises(RpcError, match=r'They sent ERROR .*: You gave bad parameters: Did not support channel_type 12,20'):
+    with pytest.raises(RpcError, match=r'They sent ERROR .*: You gave bad parameters: Did not support channel_type \[12,20\]'):
         l1.rpc.fundchannel_start(l2.info['id'], FUNDAMOUNT, channel_type=[STATIC_REMOTEKEY, ANCHORS_OLD])
 
     # Now make l2 accept it!