summaryrefslogtreecommitdiff
path: root/src/simplewallet
Commit message (Collapse)AuthorAgeFilesLines
* monero-wallet-cli wording changes 'n' stuffluigi11112016-12-121-11/+13
| | | | Possibly other pedantry. Pedants are people too.
* simplewallet: restore automatically setting trusted local daemonmoneromooo-monero2016-12-081-0/+43
|
* Merge pull request #1379Riccardo Spagni2016-12-041-7/+12
|\ | | | | | | d6086f5b Improve daemon RPC version handling (moneromooo-monero)
| * Improve daemon RPC version handlingmoneromooo-monero2016-11-261-7/+12
| | | | | | | | | | | | | | Daemon RPC version is now composed of a major and minor number, so that incompatible changes bump the major version, while compatible changes can still bump the minor version without causing clients to unnecessarily complain.
* | Added command_line::is_yesLee Clagett2016-11-251-17/+15
|/
* simplewallet: remove double confirmation when submitting signed txmoneromooo-monero2016-11-231-51/+0
|
* Merge pull request #1344Riccardo Spagni2016-11-172-16/+58
|\ | | | | | | | | | | | | | | | | | | | | | | | | a0131c8 wallet: auto sync outputs and key images in cold signing files (moneromooo-monero) f806611 wallet2: fill key image and pubkey maps when importing outputs (moneromooo-monero) 5fe363c wallet: cast indices to string in logs to be nice to CLANG (moneromooo-monero) 11ae187 wallet2: try all tx keys when scanning a new transaction (moneromooo-monero) 072d646 wallet2: fill in key image map when importing key images (moneromooo-monero) 23d80b1 core: remove any tx pubkey from extra before adding one (moneromooo-monero) d72376d simplewallet: add a verbose flag to incoming_transfers (moneromooo-monero) 47413a5 simplewallet: spell out change when signing a transfer (moneromooo-monero) 14cb088 simplewallet: print public keys too on spendkey/viewkey commands (moneromooo-monero) a99ab49 wallet: fix serialization of new m_key_image_known member (moneromooo-monero)
| * wallet: auto sync outputs and key images in cold signing filesmoneromooo-monero2016-11-162-8/+22
| | | | | | | | | | | | | | | | | | When passing around unsigned and signed transactions, outputs and key images are passed along (outputs are passed along unsigned transactions from the hot wallet to the cold wallet, key images are passed along with signed transations from the cold wallet to the hot wallet), to allow more user friendly syncing between hot and cold wallets.
| * simplewallet: add a verbose flag to incoming_transfersmoneromooo-monero2016-11-131-6/+18
| | | | | | | | Prints pubkey and key image as well
| * simplewallet: spell out change when signing a transfermoneromooo-monero2016-11-131-2/+16
| | | | | | | | Also catch change to multiple addresses, this is unexpected
| * simplewallet: print public keys too on spendkey/viewkey commandsmoneromooo-monero2016-11-131-2/+4
| |
* | simplewallet: include amount sent in transfer confirmationmoneromooo-monero2016-11-131-0/+10
|/
* Fixed generate-from-json bug introduced in 358e068Lee Clagett2016-11-121-1/+2
|
* Created monero-wallet-rpc, moving functionality from monero-wallet-cliLee Clagett2016-11-105-1035/+117
|
* wallet: encrypt outputs and key images files with the view keymoneromooo-monero2016-11-071-14/+43
| | | | | | This key is available to both cold and hot wallet. Authenticated encryption will guard against interception and/or modification of the file.
* wallet: fix bad amounts/fees againmoneromooo-monero2016-11-021-2/+2
| | | | | | | | | | | m_amount_out was sometimes getting initialized with the sum of an transaction's outputs, and sometimes with the sum of outputs that were not change. This caused confusion and bugs. We now always set it to the sum of outputs. This reverts an earlier fix for bad amounts as this used the other semantics. The wallet data should be converted automatically in a percentage of cases that I'm hesitant to estimate. In any case, restoring from seed or keys or rebuilding the cache will get it right.
* Merge pull request #1281Riccardo Spagni2016-11-012-0/+102
|\ | | | | | | bb560dd wallet: new import_outputs/export_outputs commands (moneromooo-monero)
| * wallet: new import_outputs/export_outputs commandsmoneromooo-monero2016-10-302-0/+102
| | | | | | | | | | | | | | | | The intended use is to export outputs from a hot wallet, which can scan incoming transfers from the network, and import them in the cold wallet, which can't. The cold wallet can then compute key images for those outputs, which can then be exported with export_key_images, etc.
* | Merge pull request #1277Riccardo Spagni2016-11-012-5/+17
|\ \ | | | | | | | | | | | | 7e6d3cf wallet: set incoming outputs' key image to 0 on view wallets (moneromooo-monero) 1f9e6a4 wallet: print tx overview on submit_transfer too (moneromooo-monero)
| * | wallet: print tx overview on submit_transfer toomoneromooo-monero2016-10-302-5/+17
| |/ | | | | | | | | This is on the potentially compromised wallet, but still guards against stupid mistakes.
* / simplewallet: remove destinations with zero amount in sign_transfermoneromooo-monero2016-10-301-0/+2
|/ | | | | This will happen when sending to another address, after removing the fee.
* simplewallet: log transactions to submit in submit_transfermoneromooo-monero2016-10-261-1/+1
| | | | | Not as trustworthy as this is in the view wallet, the one that's considered compromised.
* wallet: fix pre-rct cold wallet signing not splitting changemoneromooo-monero2016-10-251-6/+16
| | | | | | | | | | | | Re-creating the transaction on the cold wallet was not splitting the change, causing the transaction to be rejected by the network. This worked on testnet since amounts do not have to be split. Also add selected_transfers, which can now be saved since they're size_t rather than iterators. This allows the view wallet to properly set the sent outputs as spent and update balance. Bump transfer file version numbers to match.
* simplewallet: factor locked_transfer, and fix a few rough edgesmoneromooo-monero2016-10-231-256/+43
| | | | | | | | | | Factor locked_transfer into transfer_main, which brings various improvements for free (multiple addresses, proper detection of multiple payment ids, obeying the prompt settings). Also fix a few things, such as using uint64_t instead of int for block heights, actually checking whether getting blockchain height succeeded, etc.
* Merge pull request #1224Riccardo Spagni2016-10-221-10/+4
|\ | | | | | | | | 8231997 simplewallet: fix sweep_all misreporting sweeped amount for rct outputs (moneromooo-monero) 985f61a wallet: force 0 mixin transactions to use pre-rct txes (moneromooo-monero)
| * simplewallet: fix sweep_all misreporting sweeped amount for rct outputsmoneromooo-monero2016-10-151-10/+4
| | | | | | | | | | RingCT outputs will be 0 in the vin, so we need to get the actual amount from elsewhere.
* | Merge pull request #1220Riccardo Spagni2016-10-221-7/+43
|\ \ | | | | | | | | | e76dcdd wallet: improve error messages when not enough money for transfer (moneromooo-monero)
| * | wallet: improve error messages when not enough money for transfermoneromooo-monero2016-10-151-7/+43
| |/
* | Merge pull request #1219Riccardo Spagni2016-10-221-1/+1
|\ \ | | | | | | | | | | | | 2d7083c wallet_api: fix wrong amount in tx history (moneromooo-monero) b5f2001 simplewallet: fix wrong amount in show_transfers (moneromooo-monero)
| * | simplewallet: fix wrong amount in show_transfersmoneromooo-monero2016-10-151-1/+1
| |/
* | Merge pull request #1200Riccardo Spagni2016-10-221-53/+67
|\ \ | |/ |/| | | a3748f4 Rewrote add_wallet_create_if_needed for clearer flow and behaviour (NanoAkron)
| * Rewrote add_wallet_create_if_needed for clearer flow and behaviourNanoAkron2016-10-151-53/+67
| |
* | simplewallet: add mixin and payment id to sweep_all helpmoneromooo-monero2016-10-041-1/+1
|/
* Merge pull request #1179Riccardo Spagni2016-10-042-0/+254
|\ | | | | | | | | | | | | | | | | | | | | 714ee99 Fix description for locked_transfer (Oyvind Kvanes) 71538f3 Rename to lockblocks and add max value (Oyvind Kvanes) 68ac060 Fix locked_transfer (Oyvind Kvanes) 7d020bd Add locked_transfer (Oyvind Kvanes) d5f918a Revert transfer_main in simplewallet (Oyvind Kvanes) 3451963 Add motifications to test out locked_transfer (Oyvind Kvanes) e5e6d88 Add more information to transaction in wallet (Oyvind Kvanes) 9b8a062 Make a small test change (Oyvind Kvanes)
| * Fix description for locked_transferOyvind Kvanes2016-10-041-1/+1
| |
| * Rename to lockblocks and add max valueOyvind Kvanes2016-10-041-45/+30
| |
| * Fix locked_transferOyvind Kvanes2016-09-281-6/+3
| |
| * Add locked_transferOyvind Kvanes2016-09-281-87/+83
| |
| * Revert transfer_main in simplewalletOyvind Kvanes2016-09-282-22/+279
| |
| * Add motifications to test out locked_transferOyvind Kvanes2016-09-281-6/+18
| |
| * Add more information to transaction in walletOyvind Kvanes2016-09-281-1/+8
| |
| * Make a small test changeOyvind Kvanes2016-09-281-1/+1
| |
* | Merge pull request #1160Riccardo Spagni2016-10-042-1/+81
|\ \ | | | | | | | | | 80b4da3 wallet: wallet option to confirm transfers with no payment id (moneromooo-monero)
| * | wallet: wallet option to confirm transfers with no payment idmoneromooo-monero2016-10-012-1/+81
| |/ | | | | | | | | | | set confirm-missing-payment-id 0|1 Defaults to true.
* | Merge pull request #1139Riccardo Spagni2016-10-041-2/+2
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 01ec195 Update CMakeLists.txt (codehalo) 446ebbc Update CMakeLists.txt (codehalo) bd773e7 Update CMakeLists.txt (codehalo) 3627cea Cleanup. Dropped "bit" from bitmonero. (Randi Joseph) 2a51396 Dropped "bit" from bitmonero. (Randi Joseph) 78b13d6 Cleanup. Dropped "bit" from bitmonero. (Randi Joseph) 1e6aedb Cleanup. Dropped "bit" from bitmonero. (Randi Joseph) 9e54616 Dropped "bit" from bitmonero. (Randi Joseph)
| * | Dropped "bit" from bitmonero.Randi Joseph2016-09-261-2/+2
| |/
* | Merge pull request #1162Riccardo Spagni2016-10-041-1/+1
|\ \ | | | | | | | | | | | | e5f8642 Update minimum mixin in transfer_original help text (TedTheFicus) ea45d61 Update minimum mixin in transfer_original help text (TedTheFicus)
| * | Update minimum mixin in transfer_original help textTedTheFicus2016-10-011-1/+1
| |/ | | | | | | | | Changed the wording from "from 0 to maximum available" to "from 2 to maximum available". May I also suggest putting in a number rather than "maximum available" and also clarifying that mixin = 0 is still allowed if the user has unmixable outputs (dust), which he wants to "undust" using sweep_unmixable (comment from dEBRUYNE-1 ).
* | Merge pull request #1140Riccardo Spagni2016-10-042-26/+273
|\ \ | |/ |/| | | | | | | | | bba6af9 wallet: cold wallet transaction signing (moneromooo-monero) 9872dcb wallet: fix log confusion between bytes and kilobytes (moneromooo-monero) d9b0bf9 cryptonote_core: make extra field removal more generic (moneromooo-monero) 98f19d4 serialization: add support for serializing std::pair and std::list (moneromooo-monero)
| * wallet: cold wallet transaction signingmoneromooo-monero2016-09-272-26/+273
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This change adds the ability to create a new unsigned transaction from a watch only wallet, and save it to a file. This file can then be moved to another computer/VM where a cold wallet may load it, sign it, and save it. That cold wallet does not need to have a blockchain nor daemon. The signed transaction file can then be moved back to the watch only wallet, which can load it and send it to the daemon. Two new simplewallet commands to use it: sign_transfer (on the cold wallet) submit_transfer (on the watch only wallet) The transfer command used on a watch only wallet now writes an unsigned transaction set in a file called 'unsigned_monero_tx' instead of submitting the tx to the daemon as a normal wallet does. The signed tx file is called 'signed_monero_tx'.