estimatesmartfee RPC API result inconsistent with Bitcoin Core RPC API (fee vs feerate)
#1153
Comments
pdyraga
added a commit
to keep-network/keep-core
that referenced
this issue
Jun 15, 2023
This PR adds Kubernetes manifests of bitcoin node and electrumx server. The configuration is based on the V1's config: https://github.com/keep-network/tbtc/tree/main/infrastructure/kube/keep-test with some improvements. For V1 we were running bcoin, now we switch to bitcoind, due to bcoin-org/bcoin#1153 Bitcoind and Electrum servers are running in dedicated kubernetes namespaces: - for keep-test: `bitcoin-testnet` - for keep-prd: `bitcoin` We started with one replica for each server and after sync was done, we created snapshots that were used for PVC creation for other replicas. Closes: #3590
|
Fixed in 014a104 |
# for free
to join this conversation on GitHub.
Already have an account?
# to comment
Bitcoin Core RPC API defines the result of
estimatesmartfeemethod to return afeerateproperty (see: https://developer.bitcoin.org/reference/rpc/estimatesmartfee.html#result).While Bcoin RPC API returns a
feeproperty instead:bcoin/lib/node/rpc.js
Lines 2233 to 2236 in 92433b2
The
feeproperty should be renamed tofeerateto maintain compatibility across the clients.The text was updated successfully, but these errors were encountered: