Skip to content

Commit

Permalink
[FIXUP] Remove payment proofs from atomic swaps
Browse files Browse the repository at this point in the history
  • Loading branch information
@GeneFerneau
GeneFerneau committed May 27, 2021
1 parent b1104ad commit ecb5ffd
Show file tree
Hide file tree
Showing 10 changed files with 111 additions and 161 deletions.
166 changes: 83 additions & 83 deletions api/src/foreign_rpc.rs
View file
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,7 @@ pub trait ForeignRpc {
}
}
# "#
# ,false, 0, TxFlow::Standard);
# ,false, 0, false, TxFlow::Standard);
```
*/
fn check_version(&self) -> Result<VersionInfo, ErrorKind>;
Expand Down Expand Up @@ -107,7 +107,7 @@ pub trait ForeignRpc {
}
}
# "#
# ,false, 4, TxFlow::Standard);
# ,false, 4, false, TxFlow::Standard);
```
*/

Expand Down Expand Up @@ -182,7 +182,7 @@ pub trait ForeignRpc {
}
}
# "#
# ,false, 5, TxFlow::Standard);
# ,false, 5, true, TxFlow::Standard);
```
*/
fn receive_tx(
Expand Down Expand Up @@ -211,10 +211,6 @@ pub trait ForeignRpc {
"fee": "23500000",
"id": "0436430c-2b02-624c-2032-570501212b00",
"off": "d202964900000000d302964900000000d402964900000000d502964900000000",
"proof": {
"raddr": "32cdd63928854f8b2628b1dce4626ddcdf35d56cb7cfdf7d64cca5822b78d4d3",
"saddr": "32cdd63928854f8b2628b1dce4626ddcdf35d56cb7cfdf7d64cca5822b78d4d3"
},
"sigs": [
{
"nonce": "02b57c1f4fea69a3ee070309cf8f06082022fe06f25a9be1851b56ef0fa18f25d6",
Expand All @@ -237,7 +233,6 @@ pub trait ForeignRpc {
"result": {
"Ok": {
"amt": "6000000000",
"atomic_id": "0",
"coms": [
{
"c": "091582c92b99943b57955e52b5ccf1223780c2a2e55995c00c86fca2bcb46b6b9f",
Expand All @@ -247,31 +242,25 @@ pub trait ForeignRpc {
"fee": "23500000",
"id": "0436430c-2b02-624c-2032-570501212b00",
"off": "a4f88ac429dee1d453ae33ed9f944417a52c7310477936e484fd83f0f22db483",
"proof": {
"raddr": "32cdd63928854f8b2628b1dce4626ddcdf35d56cb7cfdf7d64cca5822b78d4d3",
"rsig": "02357a13b304ba8e22f4896d5664b72ad6d1b824e88782e2b716686ea14ec47281ef5ee14c03ead84c3260f5b0c1529ad3ddae57f28f6b8b1b66532bfcb2ee0f",
"saddr": "32cdd63928854f8b2628b1dce4626ddcdf35d56cb7cfdf7d64cca5822b78d4d3"
},
"sigs": [
{
"nonce": "02b57c1f4fea69a3ee070309cf8f06082022fe06f25a9be1851b56ef0fa18f25d6",
"xs": "023878ce845727f3a4ec76ca3f3db4b38a2d05d636b8c3632108b857fed63c96de"
},
{
"atomic": "03cefcb7f65194418ee236f01638034238a7c996cd73fa9673e0c763eab828f119",
"atomic": "03e1d14f7b440af4944193b0559452651720ecf1847ef0f7092ef7e68414b8d732",
"nonce": "031b84c5567b126440995d3ed5aaba0565d71e1834604819ff9c17f5e9d5dd078f",
"part": "8f07ddd5e9f5179cff19486034181ed76505baaad53e5d994064127b56c5841b1e25843c36d8bdcf87f208666490a16281943a6504209c3ea53fb9f1f7af0cf8",
"part": "8f07ddd5e9f5179cff19486034181ed76505baaad53e5d994064127b56c5841bf9df0f2c4d96d6bb0b2e6e0b94ad6cbbc0e3f9e5582b085b31b5c8bb0e15bd36",
"xs": "02e3c128e436510500616fef3f9a22b15ca015f407c8c5cf96c9059163c873828f"
}
],
"atomic_id": "046d7761746f6d69630000000000000000",
"sta": "A2",
"ver": "4:2"
}
}
}
# "#
# ,false, 5, TxFlow::Atomic);
# ,false, 5, true, TxFlow::Atomic);
```
*/
fn receive_atomic_tx(
Expand Down Expand Up @@ -370,7 +359,7 @@ pub trait ForeignRpc {
}
}
# "#
# ,false, 5, TxFlow::Invoice);
# ,false, 5, true, TxFlow::Invoice);
```
*/
fn finalize_tx(&self, slate: VersionedSlate) -> Result<VersionedSlate, ErrorKind>;
Expand Down Expand Up @@ -451,6 +440,7 @@ pub fn run_doctest_foreign(
test_dir: &str,
use_token: bool,
blocks_to_mine: u64,
do_tx: bool,
tx_flow: TxFlow,
) -> Result<Option<serde_json::Value>, String> {
use easy_jsonrpc_mw::Handler;
Expand Down Expand Up @@ -573,81 +563,84 @@ pub fn run_doctest_foreign(
assert!(wallet_refreshed);
}

match tx_flow {
TxFlow::Invoice => {
let amount = 60_000_000_000;
let mut slate = {
let mut w_lock = wallet2.lock();
if do_tx {
match tx_flow {
TxFlow::Invoice => {
let amount = 60_000_000_000;
let mut slate = {
let mut w_lock = wallet2.lock();
let w = w_lock.lc_provider().unwrap().wallet_inst().unwrap();
let args = IssueInvoiceTxArgs {
amount,
..Default::default()
};
api_impl::owner::issue_invoice_tx(&mut **w, (&mask2).as_ref(), args, true)
.unwrap()
};
slate = {
let mut w_lock = wallet1.lock();
let w = w_lock.lc_provider().unwrap().wallet_inst().unwrap();
let args = InitTxArgs {
src_acct_name: None,
amount: slate.amount,
minimum_confirmations: 2,
max_outputs: 500,
num_change_outputs: 1,
selection_strategy_is_use_all: true,
..Default::default()
};
api_impl::owner::process_invoice_tx(
&mut **w,
(&mask1).as_ref(),
&slate,
args,
true,
)
.unwrap()
};
println!("INIT INVOICE SLATE");
// Spit out slate for input to finalize_tx
println!("{}", serde_json::to_string_pretty(&slate).unwrap());
}
TxFlow::Standard => {
let amount = 60_000_000_000;
let mut w_lock = wallet1.lock();
let w = w_lock.lc_provider().unwrap().wallet_inst().unwrap();
let args = IssueInvoiceTxArgs {
let args = InitTxArgs {
src_acct_name: None,
amount,
minimum_confirmations: 2,
max_outputs: 500,
num_change_outputs: 1,
selection_strategy_is_use_all: true,
..Default::default()
};
api_impl::owner::issue_invoice_tx(&mut **w, (&mask2).as_ref(), args, true).unwrap()
};
slate = {
let slate =
api_impl::owner::init_send_tx(&mut **w, (&mask1).as_ref(), args, true).unwrap();
println!("INIT SLATE");
// Spit out slate for input to finalize_tx
println!("{}", serde_json::to_string_pretty(&slate).unwrap());
}
TxFlow::Atomic => {
let amount = 60_000_000_000;
let mut w_lock = wallet1.lock();
let w = w_lock.lc_provider().unwrap().wallet_inst().unwrap();
let args = InitTxArgs {
src_acct_name: None,
amount: slate.amount,
amount,
minimum_confirmations: 2,
max_outputs: 500,
num_change_outputs: 1,
selection_strategy_is_use_all: true,
..Default::default()
};
api_impl::owner::process_invoice_tx(&mut **w, (&mask1).as_ref(), &slate, args, true)
.unwrap()
};
println!("INIT INVOICE SLATE");
// Spit out slate for input to finalize_tx
println!("{}", serde_json::to_string_pretty(&slate).unwrap());
}
TxFlow::Standard => {
let amount = 60_000_000_000;
let mut w_lock = wallet1.lock();
let w = w_lock.lc_provider().unwrap().wallet_inst().unwrap();
let args = InitTxArgs {
src_acct_name: None,
amount,
minimum_confirmations: 2,
max_outputs: 500,
num_change_outputs: 1,
selection_strategy_is_use_all: true,
..Default::default()
};
let slate =
api_impl::owner::init_send_tx(&mut **w, (&mask1).as_ref(), args, true).unwrap();
println!("INIT SLATE");
// Spit out slate for input to finalize_tx
println!("{}", serde_json::to_string_pretty(&slate).unwrap());
}
TxFlow::Atomic => {
let amount = 60_000_000_000;
let mut w_lock = wallet1.lock();
let w = w_lock.lc_provider().unwrap().wallet_inst().unwrap();
let args = InitTxArgs {
src_acct_name: None,
amount,
minimum_confirmations: 2,
max_outputs: 500,
num_change_outputs: 1,
selection_strategy_is_use_all: true,
..Default::default()
};
let derive_path = 0;
let slate = api_impl::owner::init_atomic_swap(
&mut **w,
(&mask1).as_ref(),
args,
derive_path,
true,
)
.unwrap();
println!("INIT SLATE");
// Spit out slate for input to finalize_tx
println!("{}", serde_json::to_string_pretty(&slate).unwrap());
let slate =
api_impl::owner::init_atomic_swap(&mut **w, (&mask1).as_ref(), args, true)
.unwrap();
println!("INIT SLATE");
// Spit out slate for input to finalize_tx
println!("{}", serde_json::to_string_pretty(&slate).unwrap());
}
}
}

Expand All @@ -667,7 +660,7 @@ pub fn run_doctest_foreign(
#[doc(hidden)]
#[macro_export]
macro_rules! doctest_helper_json_rpc_foreign_assert_response {
($request:expr, $expected_response:expr, $use_token:expr, $blocks_to_mine:expr, $tx_flow:expr) => {
($request:expr, $expected_response:expr, $use_token:expr, $blocks_to_mine:expr, $do_tx:expr, $tx_flow:expr) => {
// create temporary wallet, run jsonrpc request on owner api of wallet, delete wallet, return
// json response.
// In order to prevent leaking tempdirs, This function should not panic.
Expand All @@ -687,9 +680,16 @@ macro_rules! doctest_helper_json_rpc_foreign_assert_response {
let request_val: Value = serde_json::from_str($request).unwrap();
let expected_response: Value = serde_json::from_str($expected_response).unwrap();

let response = run_doctest_foreign(request_val, dir, $use_token, $blocks_to_mine, $tx_flow)
.unwrap()
.unwrap();
let response = run_doctest_foreign(
request_val,
dir,
$use_token,
$blocks_to_mine,
$do_tx,
$tx_flow,
)
.unwrap()
.unwrap();

if response != expected_response {
panic!(
Expand Down
10 changes: 2 additions & 8 deletions api/src/owner.rs
View file
Original file line number Diff line number Diff line change
Expand Up @@ -801,6 +801,7 @@ where
///
/// // . . .
/// // The slate has been recieved from the invoicer, somehow
/// # use grin_wallet_libwallet::TxFlow;
/// # let slate = Slate::blank(2, TxFlow::Invoice);
/// let args = InitTxArgs {
/// src_acct_name: None,
Expand Down Expand Up @@ -866,18 +867,11 @@ where
&self,
keychain_mask: Option<&SecretKey>,
args: InitTxArgs,
derive_path: u32,
) -> Result<Slate, Error> {
let mut w_lock = self.wallet_inst.lock();
let w = w_lock.lc_provider()?.wallet_inst()?;
let send_args = args.send_args.clone();
let slate = owner::init_atomic_swap(
&mut **w,
keychain_mask,
args,
derive_path,
self.doctest_mode,
)?;
let slate = owner::init_atomic_swap(&mut **w, keychain_mask, args, self.doctest_mode)?;
if let Some(sa) = send_args {
let tor_config_lock = self.tor_config.lock();
let tc = tor_config_lock.clone();
Expand Down
29 changes: 6 additions & 23 deletions api/src/owner_rpc.rs
View file
Original file line number Diff line number Diff line change
Expand Up @@ -619,8 +619,7 @@ pub trait OwnerRpc {
"payment_proof_recipient_address": "tgrin1xtxavwfgs48ckf3gk8wwgcndmn0nt4tvkl8a7ltyejjcy2mc6nfs9gm2lp",
"ttl_blocks": null,
"send_args": null
},
"derive_path": 0
}
},
"id": 1
}
Expand All @@ -633,14 +632,9 @@ pub trait OwnerRpc {
"result": {
"Ok": {
"amt": "6000000000",
"atomic_id": "046d7761746f6d69630000000000000000",
"fee": "23000000",
"id": "0436430c-2b02-624c-2032-570501212b00",
"off": "456498224e2a6850e073ab8fb5c32fcccfe70272c61759ea4ade53ae7dc367e6",
"proof": {
"raddr": "32cdd63928854f8b2628b1dce4626ddcdf35d56cb7cfdf7d64cca5822b78d4d3",
"saddr": "32cdd63928854f8b2628b1dce4626ddcdf35d56cb7cfdf7d64cca5822b78d4d3"
},
"sigs": [
{
"nonce": "031b84c5567b126440995d3ed5aaba0565d71e1834604819ff9c17f5e9d5dd078f",
Expand All @@ -656,12 +650,8 @@ pub trait OwnerRpc {
# , 4, false, false, false, false, false);
```
*/
fn init_atomic_swap(
&self,
token: Token,
args: InitTxArgs,
derive_path: u32,
) -> Result<VersionedSlate, ErrorKind>;
fn init_atomic_swap(&self, token: Token, args: InitTxArgs)
-> Result<VersionedSlate, ErrorKind>;

/**
Networked version of [Owner::countersign_atomic_swap](struct.Owner.html#method.countersign_atomic_swap).
Expand Down Expand Up @@ -714,7 +704,6 @@ pub trait OwnerRpc {
"result": {
"Ok": {
"amt": "60000000000",
"atomic_id": "046d7761746f6d69630000000000000000",
"coms": [
{
"c": "087df32304c5d4ae8b2af0bc31e700019d722910ef87dd4eec3197b80b207e3045",
Expand All @@ -727,10 +716,6 @@ pub trait OwnerRpc {
{
"c": "099b48cfb1f80a2347dc89818449e68e76a3c6817a532a8e9ef2b4a5ccf4363850",
"p": "29701ceae262cac77b79b868c883a292e61e6de8192b868edcd1300b0973d91396b156ace6bd673402a303de10ddd8a5e6b7f17ba6557a574a672bd04cc273ab04ed8e2ca80bac483345c0ec843f521814ce1301ec9adc38956a12b4d948acce71295a4f52bcdeb8a1c9f2d6b2da5d731262a5e9c0276ef904df9ef8d48001420cd59f75a2f1ae5c7a1c7c6b9f140e7613e52ef9e249f29f9340b7efb80699e460164324616f98fd4cde3db52497c919e95222fffeacb7e65deca7e368a80ce713c19de7da5369726228ee336f5bd494538c12ccbffeb1b9bfd5fc8906d1c64245b516f103fa96d9c56975837652c1e0fa5803d7ccf1147d8f927e36da717f7ad79471dbe192f5f50f87a79fc3fe030dba569b634b92d2cf307993cce545633af263897cd7e6ebf4dcafb176d07358bdc38d03e45a49dfa9c8c6517cd68d167ffbf6c3b4de0e2dd21909cbad4c467b84e5700be473a39ac59c669d7c155c4bcab9b8026eea3431c779cd277e4922d2b9742e1f6678cbe869ec3b5b7ef4132ddb6cdd06cf27dbeb28be72b949fa897610e48e3a0d789fd2eea75abc97b3dc7e00e5c8b3d24e40c6f24112adb72352b89a2bef0599345338e9e76202a3c46efa6370952b2aca41aadbae0ea32531acafcdab6dd066d769ebf50cf4f3c0a59d2d5fa79600a207b9417c623f76ad05e8cccfcd4038f9448bc40f127ca7c0d372e46074e334fe49f5a956ec0056f4da601e6af80eb1a6c4951054869e665b296d8c14f344ca2dc5fdd5df4a3652536365a1615ad9b422165c77bf8fe65a835c8e0c41e070014eb66ef8c525204e990b3a3d663c1e42221b496895c37a2f0c1bf05e91235409c3fe3d89a9a79d6c78609ab18a463311911f71fa37bb73b15fcd38143d1404fd2ce81004dc7ff89cf1115dcc0c35ce1c1bf9941586fb959770f2618ccb7118a7"
},
{
"c": "09ede20409d5ae0d1c0d3f3d2c68038a384cdd6b7cc5ca2aab670f570adc2dffc3",
"p": "6d86fe00220f8c6ac2ad4e338d80063dba5423af525bd273ecfac8ef6b509192732a8cd0c53d3313e663ac5ccece3d589fd2634e29f96e82b99ca6f8b953645a005d1bc73493f8c41f84fb8e327d4cbe6711dba194a60db30700df94a41e1fda7afe0619169389f8d8ee12bddf736c4bc86cd5b1809a5a27f195209147dc38d0de6f6710ce9350f3b8e7e6820bfe5182e6e58f0b41b82b6ec6bb01ffe1d8b3c2368ebf1e31dfdb9e00f0bc68d9119a38d19c038c29c7b37e31246e7bba56019bc88881d7d695d32557fc0e93635b5f24deffefc787787144e5de7e86281e79934e7e20d9408c34317c778e6b218ee26d0a5e56b8b84a883e3ddf8603826010234531281486454f8c2cf3fee074f242f9fc1da3c6636b86fb6f941eb8b633d6e3b3f87dfe5ae261a40190bd4636f433bcdd5e3400255594e282c5396db8999d95be08a35be9a8f70fdb7cf5353b90584523daee6e27e208b2ca0e5758b8a24b974dca00bab162505a2aa4bcefd8320f111240b62f861261f0ce9b35979f9f92da7dd6989fe1f41ec46049fd514d9142ce23755f52ec7e64df2af33579e9b8356171b91bc96b875511bef6062dd59ef3fe2ddcc152147554405b12c7c5231513405eb062aa8fa093e3414a144c544d551c4f1f9bf5d5d2ff5b50a3f296c800907704bed8d8ee948c0855eff65ad44413af641cdc68a06a7c855be7ed7dd64d5f623bbc9645763d48774ba2258240a83f8f89ef84d21c65bcb75895ebca08b0090b40aafb7ddef039fcaf4bad2dbbac72336c4412c600e854d368ed775597c15d2e66775ab47024ce7e62fd31bf90b183149990c10b5b678501dbac1af8b2897b67d085d87cab7af4036cba3bdcfdcc7548d7710511045813c6818d859e192e03adc0d6a6b30c4cbac20a0d6f8719c7a9c3ad46d62eec464c4c44b58fca463fea3ce1fc51"
}
],
"fee": "23500000",
Expand Down Expand Up @@ -2000,11 +1985,9 @@ where
&self,
token: Token,
args: InitTxArgs,
derive_path: u32,
) -> Result<VersionedSlate, ErrorKind> {
let out_slate =
Owner::init_atomic_swap(self, (&token.keychain_mask).as_ref(), args, derive_path)
.map_err(|e| e.kind())?;
let out_slate = Owner::init_atomic_swap(self, (&token.keychain_mask).as_ref(), args)
.map_err(|e| e.kind())?;
let version = SlateVersion::V4;
Ok(VersionedSlate::into_version(out_slate, version).map_err(|e| e.kind())?)
}
Expand Down Expand Up @@ -2484,7 +2467,7 @@ pub fn run_doctest_owner(
..Default::default()
};
let mut slate = if countersign_atomic {
api_impl::owner::init_atomic_swap(&mut **w, (&mask1).as_ref(), args, 0, true).unwrap()
api_impl::owner::init_atomic_swap(&mut **w, (&mask1).as_ref(), args, true).unwrap()
} else {
api_impl::owner::init_send_tx(&mut **w, (&mask1).as_ref(), args, true).unwrap()
};
Expand Down
2 changes: 1 addition & 1 deletion api/tests/slate_versioning.rs
View file
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,7 @@ fn _receive_versioned_slate() {
let request_val: Value = serde_json::from_str(v1_req).unwrap();
let expected_response: Value = serde_json::from_str(v1_resp).unwrap();

let response = run_doctest_foreign(request_val, dir, false, 5, TxFlow::Standard)
let response = run_doctest_foreign(request_val, dir, false, 5, false, TxFlow::Standard)
.unwrap()
.unwrap();

Expand Down
Loading

0 comments on commit ecb5ffd

Please sign in to comment.