-
Notifications
You must be signed in to change notification settings - Fork 67
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
SPV with multi server validation #64
Comments
Some thoughts/questions:
|
Implemented in #59, the points that need further discussion, will be discussed in separate issues. |
The SPV with multi-server mechanism at a library level should allow:
spv_enabled
innetwork_parameters.cpp
, configurable via theconnect
methodblockstream.info:995
. It will never switch automatically. Fieldelectrum_url
and flagtls
innetwork_parameters.cpp
, configurable via theconnect
methodspv_cross_validation
innetwork_parameters.cpp
(not exist at the moment, to be added), configurable via theconnect
method.spv_cross_validation_electrum_urls
, could be overwritable fromconnect
. URLs could be initialized from electrum github and eventually updated on releases.Apps are getting the result of SPV validation directly from the
list_transactions
in thespv_verified
field which can assume the following values:Native rust could handle the population of the field with the internal store populated with the Headers thread, while the external API at the moment is triggered directly from the
list_transactions
call, to avoid to be too slow the result is cached and only onein_progress
is allowed (see impl details)The external API cache could potentially leak information of wallet txs, because of this an
encryption_key
must be passed to thespv_verify_tx
, this field should be added to theget_transactions
call.The text was updated successfully, but these errors were encountered: