3a61037733 Retry client should return AlreadySubscribed error (Riccardo Casatta)
Pull request description:
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.
Backport for version 0.14.0
ACKs for top commit:
afilini:
ACK 3a61037733
Tree-SHA512: d6542227ca03b9f3755cec05115f86de54789cfc49fc87b603b653e0458f6b03ae9fea57fa110221395a4e33cf62a5b2bd7639c4269ae5291e4c92c99ef41eba
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.
c925a9179e upgrade bitcoin dep to 0.30.0 (Riccardo Casatta)
Pull request description:
ACKs for top commit:
danielabrozzoni:
ACK c925a9179e
Tree-SHA512: 2b9b7756639af4d931213046866dc36ebee2a5adcbd01c906e27990696b07f1bcf87c2ee009a959f96b17407637801b856e8d4fb9a933fab148f22a619a0d586
80b2adeb4b Make ScriptStatus Serializable (Riccardo Casatta)
Pull request description:
it's common to cache this value client side, so making it Serializable simplify things downstream
ACKs for top commit:
afilini:
ACK 80b2adeb4b
Tree-SHA512: 9467b4f2ee24ee2a684931df31607135f44c624f65d7d3ee64b3991a06de73a07475e4d1a4b8062093500e6a7cb47c1b2ebf094596fca33bf4cc54b26851b910
40cecd5010 Use Borrow for all items in IntoIterator (Riccardo Casatta)
bf2f8ef2e5 script_subscribe IntoIterator accepts Borrow<&Script> (Riccardo Casatta)
156e6fc839 Implement batch call for script subscribe (Riccardo Casatta)
Pull request description:
script_subscribe call hasn't the batch counterpart which may be convenient for multiple subsequent subscribe.
ACKs for top commit:
afilini:
ACK 40cecd5010
Tree-SHA512: 26a3ebb9d1267a224c85b765cf793523771bf7e74990dd804a35693bfebc549f7524214725092eac2276f22cd24eafc72270986549ac02f52641db83fcb71324
561db261d3 `timeout()` and `socks()` don't return Result (Riccardo Casatta)
Pull request description:
The `Result` was necessary in the past because they couldn't both be set,
this is not true anymore after 7493630ed8
ACKs for top commit:
afilini:
ACK 561db261d3
Tree-SHA512: 7654cfbbc7beac6acec786060e373bbc046e1dae0faf2b2ca1df33121e9e92a8f4950c7f8775e44226a62483aeae41c76162cf037ce38137c44d2fedd38560c8
97e60bad4b Bump version to 0.12.0 (Alekos Filini)
Pull request description:
ACKs for top commit:
danielabrozzoni:
ACK 97e60bad4b
Tree-SHA512: e71c2093229552b5317f9309008924a6b2e8f6a48564535d2cd77e6830f342359574518836cfa579460b642c6aa8a2f36b81637b2ee8a48a68f695425333f1a2
d132572c14 Bump bitcoin dependency to ^0.29, minor related fixes (Steve Myers)
Pull request description:
This version bump for `bitcoin` dependency is required to fix https://github.com/bitcoindevkit/bdk/issues/646.
ACKs for top commit:
afilini:
ACK d132572c14
Tree-SHA512: ad89362fa32e386f02ef5721489d2c24b342af8997ceac09933dfc979f70a12b876b4856dd8982942968d8ad82ae34c49386744f51db22c70289a259a195c943
f3caf4a279 Bump version to 0.11.0 (Alekos Filini)
Pull request description:
ACKs for top commit:
danielabrozzoni:
ACK f3caf4a279
Tree-SHA512: 1b8f9853452f1462daa3788bfa3f110481d8674b870ad67eebcf8857f409312b3d69dc188ee1bc5093708f965c59c5e89fd75605bfa9059239de56fb1dbdeaee
fc8ba9de90 Use idiomatic rust for the `raw_call` method (Alekos Filini)
Pull request description:
ACKs for top commit:
danielabrozzoni:
tACK fc8ba9de90
Tree-SHA512: 1b2e92eff78df5b9fe3e6d9121af7c090e27e25aecd39187e6983252b288e0109350603d175ded2cf25365d6a85601ad4da5d8495c9943fe27280c36fe5f0c49
cbda7a22db Revert "Ignore empty lines" (Alekos Filini)
Pull request description:
This reverts commit 0e3e6325e6.
This has been causing issues with tests timing out
ACKs for top commit:
notmandatory:
utACK cbda7a22db
Tree-SHA512: ff683e16e623e71f017a361dc85e12fd7ea23d8ebd0f0616dd543aa3b12645540ae82f7eaab233dd846f5e802540064ce6a56e056d225a79e5347d730a736926
aa0b957818 Client: Drop script subscriptions to free lock (Sebastian Kung)
Pull request description:
This fixes a supposed deadlock when the reader thread receives a script subscription notification from the server before it receives a subscribing response.
I am not 100% sure if this actually fixes a deadlock, but I ran into this in a regtest setup where a bunch of subscriptions were made, while some script notifications could be processed in parallel. My test failed sporadically with the electrum client locking up completely and adding this drop made the tests consistently pass again.
ACKs for top commit:
afilini:
ACK aa0b957818, makes a lot of sense!
Tree-SHA512: b1bbea71fb56a189081b9aa602f768ef50d349a6ad6f3a3f64264259954a541fee85cad84461afeafe1ba17e11950c18fdcadb7c47a5fa1b3ce2fa2ac1cdcdc0
This fixes a supposed deadlock when the reader thread receives a script
subscription notification from the server before it receives a
subscribing response
7493630ed8 Implement timeout for socks connection (Riccardo Casatta)
Pull request description:
To achieve this we have to include a modified socks library, with supports for
timeout
Remove the error `BothSocksAndTimeout` which isn't needed anymore
ACKs for top commit:
afilini:
ACK 7493630ed8
Tree-SHA512: ec1b1eba739bd954ba7811eef603cf60f3ba29d7efcf98a9eadc8dcb9b67a49fce364760718faf7b860e78b7f9eebdc4da65783e7af47792573d32cd6c76ff10
acc05c3666 Handle EOF in the reader thread (Alekos Filini)
Pull request description:
Instead of ignoring empty lines we should return an error, because it means we are at EOF.
Fixes#70 and hopefully also #67
ACKs for top commit:
RCasatta:
ACK acc05c3666
Tree-SHA512: e92b27610c91a419cc0f71ecd77205fdaeed592b50e38c42fd8cde805773f4e9e8eeda75ba14f73002d6ae20a70986f981741e6a69d1a89cf61971c174614951
To achieve this we have to include a modified socks library, with supports for
timeout
Remove the error `BothSocksAndTimeout` which isn't needed anymore
d8e339c4c1 Update build badge to use GitHub badge (dandroid)
Pull request description:
The current build badge using Travis is not working anymore.
This PR uses the badge from GitHub instead.
ACKs for top commit:
afilini:
ACK d8e339c4c1
Tree-SHA512: c08b5dc4b0d6bc0915843d95bfcf84c62317bb8a210f219feca31ce622b2f2b1241ae7987f5c395a980c6b2b3defc48227d43e8491e1adeafadc6e71ecde0db1
3d96644a0f Add raw_call to ElectrumApi (Zoe Faltibà)
Pull request description:
In order to give more flexibility to the users of this library I added a `raw_call` method to the `ElectrumApi` trait that will allow to call any arbitrary electrum API call.
ACKs for top commit:
afilini:
ACK 3d96644a0f
Tree-SHA512: b88ece7bd9ff8121272c3b3528430e9ccb8778661b15cce276c6f67a8797a4dbf4c52a8521191942c21e8b67ebde8187fd75ac49189440b0fbb71274974b4b41
29d77de2f5 doc: s/Satoshi/Bitcoin (zkao)
Pull request description:
Unit was wrong in the docs
ACKs for top commit:
afilini:
ACK 29d77de2f5
Tree-SHA512: a172647cb3c7dc55444f89f9b05832c2b073fc6062c439719bb5c4aecc739be20592ee93111ecee491b0b37e0a0c49f7c3c331a1099bbd6070711d9f5ae0c093
aea9da234b Add `Config::builder` method (志宇)
Pull request description:
The intention is to reduce some confusion on how to manipulate `Config`.
The alternative is to make `Config` fields public.
ACKs for top commit:
afilini:
ACK aea9da234b
Tree-SHA512: 4077e821260c8f6e4bc32a66548e11b1091d384988b392c981102dc3c198a5040831c5feb13b1f7b1a88fd3890d7c44c7516b94f4c2e9cbbc551581c8c72d803
