<div dir="auto">Responded on issue but wanted to close the loop here as well. The dual-funding spec explicitly requires Segwit inputs and these are checked on receipt.</div><div dir="auto"><br></div><div dir="auto">As for single-funded PSBTs, we only recently added the ability to sanity check them, we can add a “no segwit” check there.</div><div dir="auto"><br></div><div dir="auto">Cheers</div><div dir="auto">Lisa</div><div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Apr 30, 2021 at 22:53 Rusty Russell <<a href="mailto:rusty@rustcorp.com.au">rusty@rustcorp.com.au</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Eugene Siegel <<a href="mailto:elzeigel@gmail.com" target="_blank">elzeigel@gmail.com</a>> writes:<br>
> Hi, I would ask this on IRC but for some reason I can't successfully send<br>
> messages.<br>
><br>
> Anyways,<br>
> In lnd, we did not verify funding PSBTs only used segwit inputs (but now do<br>
> as of <a href="https://github.com/lightningnetwork/lnd/pull/5165" rel="noreferrer" target="_blank">https://github.com/lightningnetwork/lnd/pull/5165</a>).  I know that<br>
> c-lightning also has PSBTs, is this something that's checked?<br>
<br>
Hmm, excellent question!<br>
<br>
There are two cases in c-lightning: one is where you provide a PSBT for<br>
a standard funding transaction, and the other is where you add an input<br>
for dual-funding.  It would be nice to sanity check the former, and<br>
require it in the spec for the latter.<br>
<br>
I've opened an issue to track this, and assigned it to Lisa:<br>
<br>
<a href="https://github.com/ElementsProject/lightning/issues/4500" rel="noreferrer" target="_blank">https://github.com/ElementsProject/lightning/issues/4500</a><br>
<br>
Thanks!<br>
Rusty.<br>
<br>
</blockquote></div></div>