aboutsummaryrefslogtreecommitdiff
path: root/src/wallet/rpcwallet.cpp
Commit message (Collapse)AuthorAgeFilesLines
* s/DOGE/DIS/gTomo Ueda2021-09-021-1/+1
|
* really s/Doge/Dis/g this timeTomo Ueda2021-09-021-13/+13
|
* really s/doge/dis/g this timeTomo Ueda2021-09-021-28/+28
|
* p2p: Reduce BIP125 replace by fee increment valueMichi Lumin2021-08-041-2/+6
|
* Add query options to listunspent rpc callPedro Branco2021-02-221-9/+37
|
* [wallet] fix typo in rpcwallet.cppIkko Ashimine2021-02-061-1/+1
| | | replacable -> replaceable
* Introduce basic Dogecoin brandingRoss Nicoll2019-03-251-67/+67
|
* Add Dogecoin current fee calculation logic (#1413)Ross Nicoll2018-09-191-3/+3
| | | Introduces 1 COIN/kb fees, rounded up to the next 1 COIN.
* Modify chain consensus parameters to be height aware (#1396)Ross Nicoll2018-09-191-1/+1
| | | | | | | | | * Modify chain consensus parameters to be height aware * Correct implementation of simplified rewards in parameters * Correct max money * Use base block version in IsSuperMajority() instead of full version * Correct mining of blocks in AuxPoW tests * Add in missing pre-AuxPoW consensus checks
* Return correct error codes in fundrawtransaction().John Newbery2017-06-051-1/+1
| | | | | | | | | | | | | | | | The fundrawtransaction() RPC was returning misleading or incorrect error codes (for example RPC_INTERNAL_ERROR when funding the transaction failed). This commit fixes those error codes: - RPC_INTERNAL_ERROR should not be returned for application-level errors, only for genuine internal errors such as corrupted data. That error code has been replaced with RPC_WALLET_ERROR. This commit also updates the test cases to explicitly test the error code. Github-Pull: #9853 Rebased-From: dab804c18a427901684ebe936b2069a97e04a268
* Return correct error codes in bumpfee().John Newbery2017-06-051-12/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The bumpfee() RPC was returning misleading or incorrect error codes (for example RPC_INVALID_ADDRESS_OR_KEY when the transaction was not BIP125 replacable). This commit fixes those error codes: - RPC_INVALID_ADDRESS_OR_KEY if an invalid address was provided: - Invalid change address given - RPC_INVALID_PARAMETER if a single (non-address/key) parameter is incorrect - confTarget and totalFee options should not both be set. - Invalid confTarget - Insufficient totalFee (cannot be less than required fee) - RPC_WALLET_ERROR for any other error - Transaction has descendants in the wallet - Transaction has descendants in the mempool - Transaction has been mined, or is conflicted with a mined transaction - Transaction is not BIP 125 replaceable - Transaction has already been bumped - Transaction contains inputs that don't belong to the wallet - Transaction has multiple change outputs - Transaction does not have a change output - Fee is higher than maxTxFee - New fee rate is less than the minimum fee rate - Change output is too small. This commit also updates the test cases to explicitly test the error code. Github-Pull: #9853 Rebased-From: 6d07c62322f60eb2702c6654e994fc353bcfcf8c
* Change bitcoin address in RPC helpaddress to an invalid address, so people ↵Marijn Stollenga2017-02-281-15/+15
| | | | | | | don't accidentally send coins there (like I did). Github-Pull: #9865 Rebased-From: 83ac719d34f98cc1d3efa6ece2b95145bcde8775
* Update sendfrom RPC help to correct coin selection misconceptionRussell Yanofsky2017-02-281-0/+3
| | | | | Github-Pull: #9840 Rebased-From: f81f0d003079b96b8c33d3dfe76f90e50344bcd8
* [wallet] Set correct metadata on bumpfee wallet transactionsRussell Yanofsky2017-02-021-0/+5
| | | | | | | | Preserve comment, order form, and account strings from the original wallet transaction. Also set fTimeReceivedIsTxTime and fFromMe fields for consistency with CWallet::CreateTransaction. The latter two fields don't influence current wallet behavior, but do record that the transaction originated in the wallet instead of coming from the network or sendrawtransaction.
* Merge #9663: [RPC] clarify listunspent amount descriptionWladimir J. van der Laan2017-02-021-1/+1
|\ | | | | | | 3eba88d clarify listunspent amount description (Gregory Sanders)
| * clarify listunspent amount descriptionGregory Sanders2017-02-011-1/+1
| |
* | rpc: bumpfee: handle errors more gracefullySuhas Daftuar2017-01-311-2/+12
| |
* | rpc: bumpfee: use correct maximum signed tx size for fee calculationSuhas Daftuar2017-01-311-4/+35
| | | | | | | | | | More accurate than simply adding one byte per input, and properly handles the case where the original transaction happened to have very small signatures
* | Merge #9615: Wallet incremental feeWladimir J. van der Laan2017-01-301-28/+44
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 4b189c1 Change bumpfee result value from 'oldfee' to 'origfee'. (Alex Morcos) 0c0c63f Introduce WALLET_INCREMENTAL_RELAY_FEE (Alex Morcos) e8021ec Use CWallet::GetMinimumFee in bumpfee (Alex Morcos) ae9719a Refactor GetMinimumFee to give option of providing targetFee (Alex Morcos) fe8e8ef [rpc] Add incremental relay fee to getnetworkinfo (Alex Morcos) 6b331e6 Fix to have miner test aware of new separate block min tx fee (Alex Morcos) de6400d Fix missing use of dustRelayFee (Alex Morcos) 5b15870 Use incrementalRelayFee for BIP 125 replacement (Alex Morcos)
| * | Change bumpfee result value from 'oldfee' to 'origfee'.Alex Morcos2017-01-261-3/+3
| | | | | | | | | | | | The result value indicates the actual fee on the transaction that was replaced. But there is an error message which uses the description 'oldfee' to refer to the original fee rate applied to the new transaction's estimated max size. It was confusing that two different uses of 'oldfee' had two different numeric values.
| * | Introduce WALLET_INCREMENTAL_RELAY_FEEAlex Morcos2017-01-261-4/+12
| | | | | | | | | | | | Have wallet's default bump value be higher than the default incrementalRelayFee to future proof against changes to incremental relay fee. Only applies when not setting the fee rate directly.
| * | Use CWallet::GetMinimumFee in bumpfeeAlex Morcos2017-01-261-17/+25
| | | | | | | | | | | | | | | | | | Use the wallet's fee calculation logic to properly clamp fee against minimums and maximums when calculating the fee for a bumpfee transaction. Unless totalFee is explictly given, in which case, manually check against min, but do nothing to adjust given fee. In all cases do a final check against maxTxFee (after adding any incremental amount).
| * | [rpc] Add incremental relay fee to getnetworkinfoAlex Morcos2017-01-201-2/+2
| | |
| * | Fix missing use of dustRelayFeeAlex Morcos2017-01-191-1/+1
| | |
| * | Use incrementalRelayFee for BIP 125 replacementAlex Morcos2017-01-191-7/+7
| | |
* | | Merge #9613: [wallet] Clarify getbalance help string to explain interaction ↵Wladimir J. van der Laan2017-01-261-5/+19
|\ \ \ | |_|/ |/| | | | | | | | | | | with bumpfee 5a00659 [wallet] Clarify getbalance help string to explain interaction with bumpfee (Russell Yanofsky)
| * | [wallet] Clarify getbalance help string to explain interaction with bumpfeeRussell Yanofsky2017-01-231-5/+19
| | | | | | | | | | | | Documentation change only, no change in behavior.
* | | Merge #9516: Bug-fix: listsinceblock: use fork point as reference for blocks ↵Wladimir J. van der Laan2017-01-231-4/+14
|\ \ \ | |/ / |/| | | | | | | | | | | | | | in reorg'd chains 7ba0a00 Testing: listsinceblock should not use orphan block height. (Karl-Johan Alm) ee5c1ce Bug-fix: listsinceblock: use closest common ancestor when a block hash was provided for a chain that was not the main chain. (Karl-Johan Alm)
| * | Bug-fix: listsinceblock: use closest common ancestor when a block hash was ↵Karl-Johan Alm2017-01-181-4/+14
| | | | | | | | | | | | provided for a chain that was not the main chain.
* | | Merge #9377: fundrawtransaction: Keep change-output keys by default, make it ↵Wladimir J. van der Laan2017-01-201-1/+7
|\ \ \ | |_|/ |/| | | | | | | | | | | | | | | | | optional c9f3062 Add fundrawtransactions new reserveChangeKey option to the release notes (Jonas Schnelli) 9eb325d [QA] Add test for fundrawtransactions new reserveChangeKey option (Jonas Schnelli) 9aa4e6a [Wallet] Add an option to keep the change address key, true by default (Jonas Schnelli)
| * | [Wallet] Add an option to keep the change address key, true by defaultJonas Schnelli2017-01-191-1/+7
| |/
* | Merge #8456: [RPC] Simplified bumpfee command.Wladimir J. van der Laan2017-01-191-9/+280
|\ \ | |/ |/| | | | | | | cc0243a [RPC] bumpfee (mrbandrews) 52dde66 [wallet] Add include_unsafe argument to listunspent RPC (Russell Yanofsky) 766e8a4 [wallet] Add IsAllFromMe: true if all inputs are from wallet (Suhas Daftuar)
| * [RPC] bumpfeemrbandrews2017-01-191-0/+258
| | | | | | | | | | | | | | | | | | | | This command allows a user to increase the fee on a wallet transaction T, creating a "bumper" transaction B. T must signal that it is BIP-125 replaceable. T's change output is decremented to pay the additional fee. (B will not add inputs to T.) T cannot have any descendant transactions. Once B bumps T, neither T nor B's outputs can be spent until either T or (more likely) B is mined. Includes code by @jonasschnelli and @ryanofsky
| * [wallet] Add include_unsafe argument to listunspent RPCRussell Yanofsky2017-01-191-9/+22
| |
* | Merge #9222: Add 'subtractFeeFromAmount' option to 'fundrawtransaction'.Wladimir J. van der Laan2017-01-121-7/+31
|\ \ | | | | | | | | | 453bda6 Add 'subtractFeeFromOutputs' option to 'fundrawtransaction'. (Chris Moore)
| * | Add 'subtractFeeFromOutputs' option to 'fundrawtransaction'.Chris Moore2016-12-131-7/+31
| | |
* | | RPC help updatedMichael Rotarius2017-01-111-20/+33
| |/ |/|
* | Update RPC argument namesJohn Newbery2017-01-101-31/+31
| |
* | rpc: Argument name consistencyWladimir J. van der Laan2017-01-051-30/+30
| | | | | | | | | | | | | | | | | | The meaning is clear from the context, and we're inconsistent here. Also save typing when using named arguments. - `bitcoinaddress` -> `address` - `bitcoinprivkey` -> `privkey` - `bitcoinpubkey` -> `pubkey`
* | rpc: Named arguments for wallet callsWladimir J. van der Laan2017-01-051-71/+71
| |
* | Mark the minconf parameter to move as ignoredPieter Wuille2017-01-041-1/+1
| |
* | updated listsinceblock rpc docsaccraze2016-12-221-0/+1
|/ | | | fixes #8758
* Add option to return non-segwit serialization via rpcGregory Sanders2016-12-051-1/+1
|
* Merge #9165: SendMoney: use already-calculated balanceWladimir J. van der Laan2016-12-051-1/+1
|\ | | | | | | ea83d00 SendMoney: use already-calculated balance (instagibbs)
| * SendMoney: use already-calculated balanceinstagibbs2016-11-151-1/+1
| |
* | Make DecodeHexTx return a CMutableTransactionPieter Wuille2016-12-021-5/+4
| |
* | Make CWalletTx store a CTransactionRef instead of inheritingPieter Wuille2016-12-021-9/+9
| |
* | Rename the remaining main.{h,cpp} to validation.{h,cpp}Matt Corallo2016-12-021-1/+1
|/
* Change all instance of 'GMT epoch' to 'Unix epoch'matthias2016-10-311-1/+1
|
* Merge #9016: Return useful error message on ATMP failureWladimir J. van der Laan2016-10-281-4/+11
|\ | | | | | | 169bdab Return useful error message on ATMP failure (instagibbs)