Christian Decker
Christian Decker
I think we can close this, the root problem being somewhere else :-)
> > @justinmoon we're looking to get an RC candidate cut this week; should this patch be considered as a candidate for inclusion? > > No. It will probably take...
Roger that, I'll be happy to shepherd it from here ^^ Thanks @justinmoon :-)
Rebased on top of `master` to get CI to rerun and ready the PR for merge.
Sorry for the false alarm @jgriffiths, didn't dig deep enough into how we use PSBT apparently. This was solely used in the communication protocol with the signer, so no stored...
Can you write a test to verify that even messages are indeed forwarded to the plugin if enabled and drop the connection otherwise? It'd also be helpful to get some...
Excellent, just saw the original issue now, that makes sense to me. Sounds like a good experimental change, with no promise that it'll graduate, but the experimental deployment will inform...
Removing milestone target for now, as this requires some work. Feel free to undraft as soon as you'd like me to re-review and assign to a milestone.
This appears to duplicate a lot of code that, when changed, will cause things to slowly drift apart. Any chance of not just extracting this but also to rewire the...
Excellent, thanks @ddustin. Want me to wait for `dualopend` to be migrated on top of `interactivetx.c` or would you like to hand this PR over as is (and create an...