aboutsummaryrefslogtreecommitdiff
path: root/src/simplewallet
Commit message (Collapse)AuthorAgeFilesLines
...
| * | 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'.
* | Merge pull request #1117Riccardo Spagni2016-09-261-1/+1
|\ \ | | | | | | | | | fc855c1 Made error message consistent with other, more helpful, message (Will Skinner)
| * | Made error message consistent with other, more helpful, messageWill Skinner2016-09-221-1/+1
| |/
* / Remove redundant "default" in set mixin help descJ Ryan Littlefield2016-09-191-1/+1
|/
* Merge pull request #1094Riccardo Spagni2016-09-181-3/+5
|\ | | | | | | | | | | 06bb692 cmake: support BUILD_SHARED_LIBS built-in option (redfish) e1c7af3 cmake: transitive deps and remove deprecated LINK_* (redfish) 54010b9 crypto: armv7: slow-hash: remove redundant source include (redfish)
| * cmake: transitive deps and remove deprecated LINK_*redfish2016-09-181-3/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Keep the immediate direct deps at the library that depends on them, declare deps as PUBLIC so that targets that link against that library get the library's deps as transitive deps. Break dep cycle between blockchain_db <-> crytonote_core. No code refactoring, just hide cycle from cmake so that it doesn't complain (cycles are allowed only between static libs, not shared libs). This is in preparation for supproting BUILD_SHARED_LIBS cmake built-in option for building internal libs as shared.
* | Merge pull request #1089Riccardo Spagni2016-09-181-10/+11
|\ \ | | | | | | | | | 368485d simplewallet: do not expose the details of tx splitting to the user (moneromooo-monero)
| * | simplewallet: do not expose the details of tx splitting to the usermoneromooo-monero2016-09-171-10/+11
| |/ | | | | | | They're confusing, as people think it reports their balance.
* / simplewallet: fix wrong amount in show_transfers for pending transactionsmoneromooo-monero2016-09-171-1/+1
|/ | | | Change was not taken into consideration
* wallet: change priority/fee to ArticMine's recommendationmoneromooo-monero2016-09-162-16/+16
| | | | | | | | | | | We keep 1, 2, 3 multipliers till the fee decrase from 0.01/kB to 0.002/kB, where we start using 1, 20, 166 multipliers. This ensures the higher multiplier will compensate for the block reward penalty when pushing past 100% of the past median. The fee-multiplier wallet setting is now rename to priority, since it keeps its [0..3] range, but maps to different multiplier values.
* Add snap packaging.Casey Marshall2016-09-111-1/+2
| | | | | | | | | | | | | | | This adds [snap](https://snapcraft.io) packaging to the project. See the link for more information on snaps. Snap packages install on all Linux distributions. On Ubuntu, snap confinement with apparmor and seccomp provide an additional layer of security. This snap sets up monerod as a systemd service, which should start immediately on install. To access the wallet CLI, simply run `monero` (/snap/bin/monero). I think it's a really quick & easy way to get started with monero. I've made some opinionated decisions in the packaging just to kick this off, but I'm happy to iterate on this stuff.
* fix remaining bitmonero and simplewallet bitsRiccardo Spagni2016-09-031-5/+5
|
* rename simplewalletRiccardo Spagni2016-09-031-0/+3
|
* simplewallet: add magic and public keys to key image export filemoneromooo-monero2016-09-011-5/+30
| | | | This ensures it's hard to mix files up
* simplewallet: make the key image export format binarymoneromooo-monero2016-09-011-18/+5
| | | | Saves on space, and on some pointless hex conversions
* wallet: fix some "may be used uninitialized" warningsmoneromooo-monero2016-08-281-8/+8
| | | | | The compiler can't always work out the _found booleans are set iff the value is initialized.
* rct amount key modified as per luigi1111's recommendationsmoneromooo-monero2016-08-281-8/+20
| | | | | | | | | This allows the key to be not the same for two outputs sent to the same address (eg, if you pay yourself, and also get change back). Also remove the key amounts lists and return parameters since we don't actually generate random ones, so we don't need to save them as we can recalculate them when needed if we have the correct keys.
* wallet: do not store signatures in the wallet cachemoneromooo-monero2016-08-281-1/+1
| | | | | Saves some substantial space. Also avoid calculating tx hashes we don't need.
* simplewallet: better check_tx_key feedbackmoneromooo-monero2016-08-281-2/+20
| | | | | Print number of confirmations if the tx is in the blockchain, or a warning if it's in the pool.
* rct: make the amount key derivable by a third party with the tx keymoneromooo-monero2016-08-281-24/+16
| | | | Scheme design from luigi1114.