aboutsummaryrefslogtreecommitdiff
path: root/src/simplewallet/simplewallet.cpp
Commit message (Collapse)AuthorAgeFilesLines
...
| * | Add option to join multisig wallet pieces togetherJollyMort2017-07-291-3/+150
| |/ | | | | | | Asks user for all the data required to merge secret keys from multisig wallets into one master wallet, which then gets full control of the multisig wallet. The resulting wallet will be the same as any other regular wallet.
* | Merge pull request #2210Riccardo Spagni2017-08-071-7/+43
|\ \ | | | | | | | | | cb0b5594 Move OpenAlias console input back from libs (moneromooo-monero)
| * | Move OpenAlias console input back from libsmoneromooo-monero2017-07-271-7/+43
| |/ | | | | | | | | | | | | Library code should definitely not ask for console input unless it's clearly an input function. Delegating the user interaction part to the caller means it can now be used by a GUI, or have a decision algorithm better adapted to a particular caller.
* | Merge pull request #2196Riccardo Spagni2017-08-071-12/+50
|\ \ | | | | | | | | | 16a55347 simplewallet: show payment ids on sign_transfer (moneromooo-monero)
| * | simplewallet: show payment ids on sign_transfermoneromooo-monero2017-08-041-12/+50
| |/ | | | | | | Integrated addresses are shown when an encrypted payment id is used
* / Make successful transfer log message more informativeJollyMort2017-07-221-1/+2
|/ | | | "Payment successfully sent" can be misleading if the TX isn't confirmed and drops from TX-pool.
* Merge pull request #2159Riccardo Spagni2017-07-191-2/+2
|\ | | | | | | 6955976b Add various readline related fixes (Jethro Grassie)
| * Add various readline related fixesJethro Grassie2017-07-091-2/+2
| | | | | | | | | | | | | | | | | | | | - Add missing unbind key - Fix colored messages - Add command completion - Preserve last command input - Fix cursor position issues - Fix trailing whitespace in commands - Synchronize set_prompt
* | Merge pull request #2135Riccardo Spagni2017-07-191-0/+2
|\ \ | |/ |/| | | 5cea3552 simplewallet: lock idle scope when sweeping (moneromooo-monero)
| * simplewallet: lock idle scope when sweepingmoneromooo-monero2017-06-301-0/+2
| | | | | | | | | | This ensures the chain and related structures can't change while we're using them
* | Merge pull request #2124Riccardo Spagni2017-07-031-12/+0
|\ \ | | | | | | | | | | | | | | | 3b599d2b wallet2: get current height from the daemon on creation (moneromooo-monero) d3bb72ff wallet2: fix infinite loop on future refresh height (moneromooo-monero) 32754784 wallet: fix refresh_from_height setting on new wallet (moneromooo-monero)
| * | wallet: fix refresh_from_height setting on new walletmoneromooo-monero2017-06-261-12/+0
| |/ | | | | | | | | | | | | The previous patch was based on a wrong premise (that the daemon height was 0 because the daemon calling code wasn't yet initialized). In fact, current height approximation was not setup for testnet. Fix this.
* | Merge pull request #2112Riccardo Spagni2017-07-031-0/+15
|\ \ | | | | | | | | | | | | | | | 76043b17 monero-wallet-cli: hang on exit in readline code (#2117) (moneromooo-monero) a73a42a6 monero-wallet-cli: hang on exit in readline code (#2117) (moneromooo-monero) be9d4f04 Fix multiline wallet cli output with readline (Jethro Grassie)
| * | Fix multiline wallet cli output with readlineJethro Grassie2017-06-251-0/+15
| |/ | | | | | | | | | | monero-wallet-cli commands which have multine output sometimes causes issues with the readline support. This patch fixes show_transfers, payments and incoming_transfers.
* | Merge pull request #2110Riccardo Spagni2017-07-031-4/+4
|\ \ | |/ |/| | | | | 7d2c89b6 Add french translation (Guillaume LE VAILLANT) 3255887f simplewallet: typo fixes (Guillaume LE VAILLANT)
| * simplewallet: typo fixesGuillaume LE VAILLANT2017-06-231-4/+4
| |
* | simplewallet: removed unneeded LOCK_IDLE_SCOPE() from check_tx_key()stoffu2017-06-221-2/+0
| |
* | simplewallet: replace assert(m_wallet) with error outstoffu2017-06-221-5/+25
| |
* | Signature proving payment to destination by only revealing key derivation, ↵stoffu2017-06-221-15/+217
|/ | | | not the actual tx secret key
* Merge pull request #2081Riccardo Spagni2017-06-181-0/+14
|\ | | | | | | b2319a03 simplewallet: new command to generate a random payment id (moneromooo-monero)
| * simplewallet: new command to generate a random payment idmoneromooo-monero2017-06-101-0/+14
| | | | | | | | | | This is trivial, but often requested, and possibly hard to do in Windows. That makes it more user friendly.
* | add simplewallet --generate-new-wallet arg checksMichael Shick2017-06-071-0/+9
|/ | | | | | | Add fail-fast paths that reduce frustration around the misuse of --wallet-file combined with --restore-deterministic-wallet. Flow now gives more descriptive errors and avoids having users type in their whole seed before the failure condition is noticed.
* Merge pull request #2053Riccardo Spagni2017-05-301-10/+13
|\ | | | | | | 05f3dcf7 simplewallet: fix refresh height for new wallets (moneromooo-monero)
| * simplewallet: fix refresh height for new walletsmoneromooo-monero2017-05-281-10/+13
| | | | | | | | | | | | | | With the recent change to wallet creation code, the code was calling to the daemon before the wallet's daemon address was initialized, and thus failing. This was causing all new wallets to refresh from 0 instead of just fetching early block hashes.
* | corrected typo in help regarding min-outputs-*stoffu2017-05-091-1/+1
|/
* wallet: add sweep_below functionmoneromooo-monero2017-04-241-2/+24
| | | | | | | It sweeps all outputs below the given threshold This is available via the existing sweep_all RPC, by setting amount_threshold the desired amount (in atomic units)
* Merge pull request #1984Riccardo Spagni2017-04-241-7/+22
|\ | | | | | | | | a255a540 simplewallet: allow multiline seed (moneromooo-monero) 5fce581c mnemonics: ignore multiple whitespace between words (moneromooo-monero)
| * simplewallet: allow multiline seedmoneromooo-monero2017-04-161-7/+22
| | | | | | | | | | People are likely to enter it in three lines as it is how it is displayed at creation time
* | wallet cli: removed some inappropriate return-falsekenshi842017-04-121-16/+6
|/
* Merge pull request #1949Riccardo Spagni2017-04-111-9/+9
|\ | | | | | | | | | | 0ee018b4 wallet2: do not go over the target tx size if many destinations (moneromooo-monero) 9ae566d0 simplewallet: fix cold signing of split transactions (moneromooo-monero) aae14a10 simplewallet: allow setting confirm-missing-payment-id in watch wallets (moneromooo-monero)
| * simplewallet: fix cold signing of split transactionsmoneromooo-monero2017-04-021-3/+9
| | | | | | | | | | | | | | They'd be rejected as suspicious as the change goes to more than one destination. However, split transactions will most likely include fake zero amount change to random addresses, so we only consider change with non zero amount for this.
| * simplewallet: allow setting confirm-missing-payment-id in watch walletsmoneromooo-monero2017-04-021-6/+0
| | | | | | | | These can create transactions, even though they cannot sign them.
* | Merge pull request #1940Riccardo Spagni2017-04-111-4/+17
|\ \ | | | | | | | | | 17c7c62d Add per transaction fee to transfer command (Antti Keränen)
| * | Add per transaction fee to transfer commandAntti Keränen2017-04-011-4/+17
| |/ | | | | | | | | Allows priority argument default/unimportant/normal/elevated/priority to be used per transaction in CLI wallet's transfer command. Resolves #1913.
* | Merge pull request #1931Riccardo Spagni2017-04-111-1/+0
|\ \ | | | | | | | | | 89e100bc wallet cli: remove redundant password confirm in donate command (kenshi84)
| * | wallet cli: remove redundant password confirm in donate commandkenshi842017-03-271-1/+0
| |/
* / simplewallet: factor the per-variable code in set_variablemoneromooo-monero2017-03-261-170/+28
|/
* wallet: option to merge destinationsmoneromooo-monero2017-03-251-1/+26
| | | | | | | | | | | | With the change from the original transfer method to the new algorithm, payments to the same destination were merged. It seemed like a good idea, optimizing space. However, it is a useful tool for people who want to split large outputs into several smaller ones (ie, service providers making frequent payments, and who do not like a large chunk of their balance being locked for 10 blocks after each payment). Default to off, which is a change from the previous behavior.
* Merge pull request #1921Riccardo Spagni2017-03-251-1/+1
|\ | | | | | | | | 2bee92db Fixed typo in v5 hard fork finalized date (Derek Zhang) d17179d1 Fixed typo in CLI wallet help message (Derek Zhang)
| * Fixed typo in CLI wallet help messageDerek Zhang2017-03-241-1/+1
| |
* | wallet: try to save large outputs when using an unneeded second inputmoneromooo-monero2017-03-241-1/+65
|/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | When a single input is enough to satisfy a transfer, the code would previously try to add a second input, to match the "canonical" makeup of a transaction with two inputs and two outputs. This would cause wallets to slowly merge outputs till all the monero ends up in a single output, which causes trouble when making two transactions one after the other, since change is locked for 10 blocks, and an increasing portion of the remaining balance would end up locked on each transaction. There are two new settings (min-output-count and min-output-value) which can control when to stop adding such unneeded second outputs. The idea is that small "dust" outputs will still get added, but larger ones will not. Enable with, eg: set min-output-count 10 set min-output-value 30 to avoid using an unneeded second output of 30 monero or more, if there would be less than 10 such outputs left. This does not invalidate any other reason why such outputs would be used (ie, when they're really needed to satisfy a transfer, or when randomly picked in the normal course of selection). This may be improved in the future.
* Transfer helpAkademik Prokhor Zakharov2017-03-221-2/+2
| | | | | New pull request because I couldn't figure out how to change the previous one. 1. For clarity, I want to focus the help text for the 'transfer' command on the most typical use case (a single payment). 2. New users will prefer to use 'transfer', so the older method 'transfer_original' should refer to 'transfer' rather than the other way around.
* wallet: fix set_log not handling 0,xxx style settingsmoneromooo-monero2017-03-211-15/+1
|
* Merge pull request #1886Riccardo Spagni2017-03-181-5/+0
|\ | | | | | | f9a2b527 simplewallet: allow setting tx priority in watch wallets (moneromooo-monero)
| * simplewallet: allow setting tx priority in watch walletsmoneromooo-monero2017-03-181-5/+0
| | | | | | | | Those can create unsigned transactions
* | Merge pull request #1883Riccardo Spagni2017-03-181-2/+2
|\ \ | | | | | | | | | d0238313 use const references in catch blocks (moneromooo-monero)
| * | use const references in catch blocksmoneromooo-monero2017-03-181-2/+2
| |/
* / simplewallet: catch case where no transactions are mademoneromooo-monero2017-03-181-1/+7
|/ | | | | | This would otherwise be a silent noop, which is confusing. This can happen if the daemon is started, but not yet ready to service all requests, and this is a safe catch all.
* Add intervening v5 fork for increased min block sizemoneromooo-monero2017-03-151-6/+6
| | | | | | | | | | Minimum mixin 4 and enforced ringct is moved from v5 to v6. v5 is now used for an increased minimum block size (from 60000 to 300000) to cater for larger typical/minimum transaction size. The fee algorithm is also changed to decrease the base per kB fee, and add a cheap tier for those transactions which we do not care if they get delayed (or even included in a block).
* wallet: add password commandmoneromooo-monero2017-03-051-3/+4
| | | | | Also tweak wallet2 password code to verify password without saying it's a new wallet, because it's assuming things.