StratisBitcoinFullNode icon indicating copy to clipboard operation
StratisBitcoinFullNode copied to clipboard

[RPC] Missing implementation for RPC calls

Open fshutdown opened this issue 5 years ago • 4 comments

Following is a list of required RPC which are required in SBFN

Results for node at address: 192.168.98.200:16174

  • [Optional ] GetInfo: Pass
  • GetBlockCount: Pass
  • GetTransaction: !!!!!! Fail (The remote server returned an error: (404) Not Found.) !!!!!
  • GetRawTransaction: Pass
  • DecodeRawTransaction: Pass
  • ValidateAddress: Pass
  • GetBlockHash: Pass
  • GetBlock: Pass
  • GetBalance: !!!!!! Fail (The remote server returned an error: (404) Not Found.) !!!!!
  • ListUnspent: !!!!!! Fail (The remote server returned an error: (404) Not Found.) !!!!!
  • ListAddressGroupings: !!!!!! Fail (The remote server returned an error: (404) Not Found.) !!!!!
  • SendMany: !!!!!! Fail (The remote server returned an error: (404) Not Found.) !!!!!

Missing RPC calls details gettransaction <txid>

Get detailed information about <txid>

getbalance [account] [minconf=1]

If [account] is not specified, returns the server's total available balance. If [account] is specified, returns the balance in the account.

listunspent [minconf=1] [maxconf=9999999] ["address",...]

Returns array of unspent transaction outputs with between minconf and maxconf (inclusive) confirmations. Optionally filtered to only include txouts paid to specified addresses. Results are an array of Objects, each of which has: {txid, vout, scriptPubKey, amount, confirmations}

listaddressgroupings

Lists groups of addresses which have had their common ownership made public by common use as inputs or as the resulting change in past transactions

sendmany <fromaccount> {address:amount,...} [minconf=1] [comment]

amounts are double-precision floating point numbers

fshutdown avatar Nov 14 '18 17:11 fshutdown

Added script to test the RPC methods: TestRPC.zip

fshutdown avatar Nov 14 '18 17:11 fshutdown

getbalance should be there. Here is a test https://github.com/stratisproject/StratisBitcoinFullNode/blob/master/src/Stratis.Bitcoin.IntegrationTests/RPC/RPCTestsMutable.cs#L18-#L35

This RPC is in the wallet feature, is that included on the node?

mikedennis avatar Nov 14 '18 18:11 mikedennis

Ditto for gettransaction It's also already in the wallet feature and requires a wallet.

mikedennis avatar Nov 17 '18 14:11 mikedennis

I just want to mention, that #1761 has some comments regarding "getwalletinfo", which is not mentioned in this issue.

sondreb avatar Mar 17 '19 13:03 sondreb