-
Notifications
You must be signed in to change notification settings - Fork 107
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
getrawtransaction JSON-RPC method #3145
Labels
lightwalletd
any work associated with lightwalletd
Comments
33 tasks
Hey team! Please add your planning poker estimate with ZenHub @conradoplg @dconnolly @gustavovalverde @jvff @oxarbitrage @teor2345 @upbqdn |
@ftm1000 @conradoplg this ticket doesn't depend on the transaction database indexes. It will perform poorly until we do #3151, but it should be ok for testing. |
3 tasks
This was referenced Mar 17, 2022
6 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Motivation
lightwalletd
uses thegetrawtransaction
JSON-RPC method in response to wallet queries.So this RPC needs to be fast:
Assumptions
lightwalletd
doesn't need witnessed mempool transaction IDs for Orchard (UnminedTxId
).https://discord.com/channels/809218587167293450/809250822579028008/943326658042163270
Required Fields
The method is documented here: https://zcash.github.io/rpc/getrawtransaction.html
lightwalletd
uses these arguments:txid
andverbosity = 0
: https://github.com/zcash/lightwalletd/blob/master/frontend/service.go#L431txid
andverbosity = 1
: https://github.com/zcash/lightwalletd/blob/master/frontend/service.go#L247And only these fields are used by
lightwalletd
:Field list:
Tasks
lightwalletd
API Reference
We plan to use jsonrpc_core with:
Example Code
Here are examples of:
The text was updated successfully, but these errors were encountered: