Remove Sled crypto store, use SQLite by default

turt2live / matrix-bot-sdk

TypeScript/JavaScript SDK for Matrix bots

MIT License

194 stars 68 forks source link

Remove Sled crypto store, use SQLite by default #330

Open AndrewFerr opened 1 year ago

AndrewFerr commented 1 year ago

on account of it being removed from the crypto-sdk

Signed-off-by: Andrew Ferrazzutti andrewf@element.io

Checklist

[x] Tests written for all new code
[x] Linter has been satisfied
[x] Sign-off given on the changes (see CONTRIBUTING.md)

turt2live commented 1 year ago

I'm honestly more inclined to replace the rust-sdk with something else. These zero-notice and breaking changes are becoming more hassle than it's worth.

A migration path needs to be present for this SDK to drop sled, and the SDK will need to automatically do it in order to avoid a breaking change itself.

AndrewFerr commented 1 year ago

A migration path needs to be present for this SDK to drop sled, and the SDK will need to automatically do it in order to avoid a breaking change itself.

I've opened an issue to track this: https://github.com/vector-im/matrix-bot-sdk/issues/26

turt2live commented 11 months ago

It looks like the rust-sdk bindings have stablized a bit since this was opened, which is great. I'm tempted to land this in a v1.0.0 of the bot-sdk, as it's a breaking change.