Setup with sh(sortedmulti()) script

[notmandatory]

How do I setup a custom 2-3 sh(sortedmulti(2,key1,key2,key3)) descriptor ? This is the format that Unchained uses and I haven't been able to set it up manually in bitcoin-safe. When I enter the descriptor and keys manually the descriptor gets switched to sh(wsh(sortedmulti(2,key1,key2,key3))).

bitcoin-safe version: 1.0.0
OS: MacOS

[andreasgriffin]

Will check it asap.

[andreasgriffin]

The short answer is: Bitcoin Safe does not support creating a p2sh multisig, because the fee burden for p2sh multisig is substantial. Perhaps I should reconsider adding support for importing older wallets.

In any case: Clearly, Bitcoin Safe should prevent (and warn) you from copying unsupported descriptor types in the edit box. --> I will create a PR for this.

[notmandatory]

Agree these legacy p2sh script types are not ideal. Hopefully Unchained will be able to upgrade their software soon.
cc: @bucko13

[andreasgriffin]

I am testing and so far, I can only reproduce that Bitcoin Safe behaves correctly, in NOT doing anything with values of a descriptor type it doesn't support. (A warning message would be helpful though).

After discussion, it turns out that a warning message is important here to inform the user, that a copy&pasted descriptor was NOT accepted.

[bucko13]

Agree these legacy p2sh script types are not ideal. Hopefully Unchained will be able to upgrade their software soon. cc: @bucko13

An "upgrade" in that unchained moves to segwit completely is not in the cards. We'll support legacy indefinitely. However adding p2wsh support is very much in the cards and should be released soon. Other software we maintain (for example https://github.com/caravan-bitcoin/caravan) has supported p2wsh wallets for years though.