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
Add GetMixins to mobilecoind API #680
Add GetMixins to mobilecoind API #680
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good so far!
mobilecoind/src/payments.rs
Outdated
/// Create a TxProposal that attempts to merge multiple UTXOs into a single larger UTXO. | ||
/// | ||
/// # Arguments | ||
/// * `monitor_id` - ??? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe this generates a transaction that joins a bunch of UTXOs belonging to monitor_id+subaddress_index
mobilecoind/src/payments.rs
Outdated
/// Create a TxProposal that sends the total value of all inputs minus the fee to a single receiver. | ||
/// | ||
/// # Arguments | ||
/// * `account_key` - |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The account key the UTXOs belong to
mobilecoind/src/payments.rs
Outdated
/// * `fee` - Transaction fee, in picoMOB. | ||
/// * `from_account_key` - ??? | ||
/// * `change_subaddress` - ??? | ||
/// * `destinations` - ??? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The outputs of the transaction
mobilecoind/src/payments.rs
Outdated
/// * `from_account_key` - ??? | ||
/// * `change_subaddress` - ??? | ||
/// * `destinations` - ??? | ||
/// * `tombstone_block` - ??? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tombstone block of the transaction
mobilecoind/src/payments.rs
Outdated
/// * `change_subaddress` - ??? | ||
/// * `destinations` - ??? | ||
/// * `tombstone_block` - ??? | ||
/// * `fog_pubkey_resolver` - ??? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Used to obtain fog report key when fog support is enabled.
mobilecoind/src/payments.rs
Outdated
/// * `inputs` - UTXOs to spend, with membership proofs. | ||
/// * `rings` - A set of mixins for each input, with membership proofs. | ||
/// * `fee` - Transaction fee, in picoMOB. | ||
/// * `from_account_key` - ??? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Account key the inputs belong to and change is sent to
mobilecoind/src/payments.rs
Outdated
/// * `rings` - A set of mixins for each input, with membership proofs. | ||
/// * `fee` - Transaction fee, in picoMOB. | ||
/// * `from_account_key` - ??? | ||
/// * `change_subaddress` - ??? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Subaddress to send change to
mobilecoind/src/service.rs
Outdated
.map_err(|err| rpc_internal_error("unspent_tx_out.try_from", err, &self.logger)) | ||
// UnspentTxOut::try_from(tx_out) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Leftovers
Ok(Json(response)) | ||
} | ||
|
||
#[post("/tx-out/mixin", format = "json", data = "<request>")] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
mixins intead of mixin?
@eranrund May JsonBlockDetailsResponse return full data, like JsonTxOut, key images, etc? |
Motivation
Adds API endpoint for getting TxOuts for use as mixins.
In this PR