search

bitcoindevkit / rust-electrum-client

Bitcoin Electrum client library. Supports plaintext, TLS and Onion servers.
MIT License
78 stars 63 forks source link

Retry client should return AlreadySubscribed error #106

Closed RCasatta closed 1 year ago

RCasatta commented 1 year ago

Users may leverage the Error to avoid remembering client-side the subscription status. eg. Always subscribing and calling script_pop in case AlreadySubscribed is returned.

LeoComandini commented 1 year ago

ACK 26738931b9fb6eb484e3d4a7bb9f8a3ade8d6938, code review and tested locally.

This makes the comment here "Returns Error::AlreadySubscribed if already subscribed to the script." correct.

danielabrozzoni commented 1 year ago

Closing as duplicated of #107