rust-lightning icon indicating copy to clipboard operation
rust-lightning copied to clipboard

Simplify serialization where backwards compat. to LDK prior 0.0.113 kept us from doing so

Open tnull opened this issue 2 years ago • 7 comments

In https://github.com/lightningdevkit/rust-lightning/pull/2708 we broke compatibility with serialization prior to 0.0.113.

We should explore what parts of our serialization can be simplified now that we don't need to provide backwards compat prior to 0.0.113 anymore.

tnull avatar Nov 10 '23 08:11 tnull

I don't see breaking compatibility discussed in #2708. Was there a discussion and analysis of the pros/cons to doing so?

moneyball avatar Nov 10 '23 18:11 moneyball

There was some discussion on Discord, but we're not breaking backwards compatibility, only forwards compatibility. ie you can upgrade from 0.0.112 to the latest, but you can't upgrade then downgrade to 0.0.112. We didn't discuss it in detail, but 0.0.113 is nearly a year old, and the failure will be explicit, so it seemed fine.

TheBlueMatt avatar Nov 11 '23 20:11 TheBlueMatt

There's no rush for this.

TheBlueMatt avatar Nov 28 '23 22:11 TheBlueMatt

Dropping the milestone entirely cause there's no reason to do this at any specific rate. Also there's no specific changes outlined here so hard to know when its "done".

TheBlueMatt avatar Oct 13 '24 17:10 TheBlueMatt

Dropping the milestone entirely cause there's no reason to do this at any specific rate. Also there's no specific changes outlined here so hard to know when its "done".

if there is no user I agree that there is not reason to fix this. However, would be nice to ensure that you can upgrade and downgrade from one aversion to another to make full node implementation easy. However, this kind of upgrade will depends also by the user application

vincenzopalazzo avatar Oct 14 '24 15:10 vincenzopalazzo

We absolutely support that, in this case we broke downgrades if you downgrade almost a year of versions. In general we absolutely support downgrades at least one or two versions and try to support more, but we are willing to break it after a while.

TheBlueMatt avatar Oct 15 '24 15:10 TheBlueMatt

Post-#3604 we even removed the ability to read objects written by LDK 0.0.113. Thus, we can now go further!

TheBlueMatt avatar Mar 03 '25 19:03 TheBlueMatt