Allow arbitrary length path_id in encrypted_data

BOLT 4 allows the path_id in encrypted_data to be of an arbitrary
length, but the code currently only allows 32 bytes. Update it to match
the specification.

Author: jkczyz

lightning/src/blinded_path/mod.rs

@@ -208,7 +208,7 @@ pub(crate) struct ReceiveTlvs {
 	/// If `path_id` is `Some`, it is used to identify the blinded path that this onion message is
 	/// sending to. This is useful for receivers to check that said blinded path is being used in
 	/// the right context.
-	pub(super) path_id: Option<[u8; 32]>,
+	pub(super) path_id: Option<Vec<u8>>,
 }
 
 impl Writeable for ForwardTlvs {
@@ -226,7 +226,7 @@ impl Writeable for ReceiveTlvs {
 	fn write<W: Writer>(&self, writer: &mut W) -> Result<(), io::Error> {
 		// TODO: write padding
 		encode_tlv_stream!(writer, {
-			(6, self.path_id, option),
+			(6, self.path_id, (option, encoding: WithoutLength)),
 		});
 		Ok(())
 	}

lightning/src/onion_message/messenger.rs

@@ -323,7 +323,7 @@ where
 	}
 
 	fn respond_with_onion_message<T: CustomOnionMessageContents>(
-		&self, response: OnionMessageContents<T>, path_id: Option<[u8; 32]>,
+		&self, response: OnionMessageContents<T>, path_id: Option<Vec<u8>>,
 		reply_path: Option<BlindedPath>
 	) {
 		let sender = match self.node_signer.get_node_id(Recipient::Node) {

lightning/src/onion_message/packet.rs

@@ -275,13 +275,13 @@ impl Readable for ControlTlvs {
 		let mut _padding: Option<Padding> = None;
 		let mut _short_channel_id: Option<u64> = None;
 		let mut next_node_id: Option<PublicKey> = None;
-		let mut path_id: Option<[u8; 32]> = None;
+		let mut path_id: Option<Vec<u8>> = None;
 		let mut next_blinding_override: Option<PublicKey> = None;
 		decode_tlv_stream!(&mut r, {
 			(1, _padding, option),
 			(2, _short_channel_id, option),
 			(4, next_node_id, option),
-			(6, path_id, option),
+			(6, path_id, optional_vec),
 			(8, next_blinding_override, option),
 		});