Rusty Russell

Trivial rebase, should kick CI. Ack https://github.com/ElementsProject/lightning/commit/be29c1fb3ed821a206ad4947cef6f5471a13ba39

> I haven't looked at the DF proposal, so I can't comment on it. But regarding the single-funder case, we were mostly concerned about a miner malleating a signature from...

OK, first, our message is backwards: "We've send shutdown, waiting for theirs" (which, shown above, they never send!). Now I can't see from their error message what message they're upset...

Gah! `\|` is not valid JSON (jq agrees). OK, so we add two, and it's now valid JSON, but it's not a valid rune restriction. Needs *unescaping*.... yech...

Think I hit the same issue, once I fixed the above! I'll get your review on the fix...

> > Ack [68b4e4f](https://github.com/ElementsProject/lightning/commit/68b4e4fcf063b6b5502ccdfb6afd1ab19343ebfc) > > Thanks! This is my first PR, so I have a couple basic questions: > > 1. What does it mean when one commit is...

The peer is sending "link failed to shutdown" error messages. I don't know what that means though: some LND error message?

Yes, it will only return once the close has succeeded. That can take a *long* time with a peer doing weird stuff like this! However, this is exactly what *notifications*...

This is problematic :( We parse the options straight into the plugin; if it goes away, we lose the memory of those options. If the plugin comes back *and no...

... but commands that have a config file *definitely* remember those options! It's not impossible, it just requires some re-engineering. It means properly recalling options even after the plugin has...