--- Log opened Tue Jul 05 00:00:12 2022
01:26 < darosior> For the script parser i can just rule out scripts whose CScript size is larger than 3600 bytes as it's necessarily going to be an invalid Miniscript
01:27 < darosior> For the string parser, i used your suggestion with a very simple upper bound: if more than 3600 fragments were "opened", fail since any fragment is at *least* 1 byte.
01:55 < darosior> Added it as a new commit in https://github.com/bitcoin/bitcoin/pull/25540.
04:59 < stevenroose> andytosh1: what happened to the miniscript::descriptor::from_txin_with_witness_stack method? It was removed in a commit by you named "cleanups"
04:59 < stevenroose> But I can't seem to find an alternative API for that atm. That's from v0.12
05:02 < stevenroose> Ah it seems the new API might be in the Interpreter type
05:56 -!- andytosh1 is now known as andytoshi
06:29 < andytoshi> stevenroose: yeah, we rewrote the interpreter API a few times, i'm not sure what the current workflow is
06:29 < andytoshi> but the Descriptor::from_txin_with_witness_stack method wasn't gonna survive taproot no matter what we did
06:29 < stevenroose> fair point
06:41 < andytoshi> are we using it in liquid?
15:12 < stevenroose> yeah
15:12 < stevenroose> andytoshi: well, not yet
23:57 -!- midnight [~midnight@user/midnight] has quit [Remote host closed the connection]
23:58 -!- midnight [~midnight@user/midnight] has joined ##miniscript
--- Log closed Wed Jul 06 00:00:13 2022