summaryrefslogtreecommitdiff
path: root/tests
Commit message (Collapse)AuthorAgeFilesLines
* unit_tests: fix strtoul unit testjeffro2562024-03-071-1/+0
|
* Merge pull request #9158luigi11112024-02-241-5/+34
|\ | | | | | | 33e3f72 serialization: fix infinite loops and clean up dispatching (jeffro256)
| * serialization: fix infinite loops and clean up dispatchingjeffro2562024-02-081-5/+34
| | | | | | | | Resolves #8687
* | Merge pull request #9145luigi11112024-02-241-0/+1
|\ \ | | | | | | | | | f5b8634 Add <cstdint> to aligned test (Lee *!* Clagett)
| * | Add <cstdint> to aligned testLee *!* Clagett2024-01-291-0/+1
| |/
* | Merge pull request #9142luigi11112024-02-241-42/+37
|\ \ | | | | | | | | | 98ee46f Disable/fix ports with I2P (Lee Clagett)
| * | Disable/fix ports with I2PLee Clagett2024-01-301-42/+37
| |/
* / wallet: feature: transfer amount with fee includedjeffro2562024-02-202-0/+138
|/ | | | | | | | | | | | | | | | | To transfer ~5 XMR to an address such that your balance drops by exactly 5 XMR, provide a `subtractfeefrom` flag to the `transfer` command. For example: transfer 76bDHojqFYiFCCYYtzTveJ8oFtmpNp3X1TgV2oKP7rHmZyFK1RvyE4r8vsJzf7SyNohMnbKT9wbcD3XUTgsZLX8LU5JBCfm 5 subtractfeefrom=all If my walet balance was exactly 30 XMR before this transaction, it will be exactly 25 XMR afterwards and the destination address will receive slightly less than 5 XMR. You can manually select which destinations fund the transaction fee and which ones do not by providing the destination index. For example: transfer 75sr8AAr... 3 74M7W4eg... 4 7AbWqDZ6... 5 subtractfeefrom=0,2 This will drop your balance by exactly 12 XMR including fees and will spread the fee cost proportionally (3:5 ratio) over destinations with addresses `75sr8AAr...` and `7AbWqDZ6...`, respectively. Disclaimer: This feature was paid for by @LocalMonero.
* Merge pull request #9080luigi11112024-01-181-0/+7
|\ | | | | | | 47d8899 Fix missing checks for IsObject in ZMQ jsonrpc reading (Lee Clagett)
| * Fix missing checks for IsObject in ZMQ jsonrpc readingLee Clagett2023-11-281-0/+7
| |
* | add a test for the long term weight cacheBoog9002023-10-021-0/+35
| |
* | Merge pull request #8977luigi11112023-09-141-84/+45
|\ \ | | | | | | | | | 7dbb14b functional_tests: fix multisig tests noutputs assertion (jeffro256)
| * | functional_tests: fix multisig tests noutputs assertionjeffro2562023-08-191-84/+45
| |/ | | | | | | The changes to the multisig tests in #8914 and #8904 affected each other, this PR cleans up the code and fixes that issue.
* / wallet2: fix `store_to()` and `change_password()`jeffro2562023-08-235-8/+269
|/ | | | | | | | Resolves #8932 and: 2. Not storing cache when new path is different from old in `store_to()` and 3. Detecting same path when new path contains entire string of old path in `store_to()` and 4. Changing your password / decrypting your keys (in this method or others) and providing a bad original password and getting no error and 5. Changing your password and storing to a new file
* Merge pull request #8942luigi11112023-08-171-9/+146
|\ | | | | | | 78348bc wallet-rpc: restore from multisig seed (jeffro256)
| * wallet-rpc: restore from multisig seedjeffro2562023-08-101-9/+146
| |
* | Merge pull request #8891luigi11112023-08-171-0/+10
|\ \ | |/ |/| | | 842478c core_rpc_server: return ID of submitted block (jeffro256)
| * core_rpc_server: return ID of submitted blockjeffro2562023-06-301-0/+10
| |
* | Merge pull request #8905luigi11112023-07-061-0/+101
|\ \ | | | | | | | | | dc24312 wallet: respect frozen key images in multisig wallets [RELEASE] (jeffro256)
| * | wallet: respect frozen key images in multisig wallets [RELEASE]jeffro2562023-06-121-0/+101
| |/ | | | | | | | | | | | | Before this change, if a multisig peer asked you to sign a transaction with a frozen enote, the wallet will do it without any error or warning. This change makes it so that wallets will refuse to sign multisig transactions with frozen enotes. Disclaimer: This PR was generously funded by @LocalMonero.
* | Merge pull request #8878luigi11112023-06-272-0/+51
|\ \ | | | | | | | | | 8dc4abd common: do not use DNS to determine if address is local (tobtoht)
| * | common: do not use DNS to determine if address is localtobtoht2023-05-252-0/+51
| |/ | | | | | | Co-authored-by: j-berman <justinberman@protonmail.com>
* | Merge pull request #8566luigi11112023-06-273-8/+224
|\ \ | |/ |/| | | 65e13db wallet2: fix rescanning tx via scan_tx (j-berman)
| * wallet2: fix rescanning tx via scan_txj-berman2023-03-133-8/+224
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Detach & re-process txs >= lowest scan height - ensures that if a user calls scan_tx(tx1) after scanning tx2, the wallet correctly processes tx1 and tx2 - if a user provides a tx with a height higher than the wallet's last scanned height, the wallet will scan starting from that tx's height - scan_tx requires trusted daemon iff need to re-process existing txs: in addition to querying a daemon for txids, if a user provides a txid of a tx with height *lower* than any *already* scanned txs in the wallet, then the wallet will also query the daemon for all the *higher* txs as well. This is likely unexpected behavior to a caller, and so to protect a caller from revealing txid's to an untrusted daemon in an unexpected way, require the daemon be trusted.
* | verRctNonSemanticsSimpleCached: fix fragilityJeffrey Ryan2023-03-173-0/+427
|/
* Merge pull request #8649luigi11112023-01-111-0/+11
|\ | | | | | | 2c24322 DNSResolver: fix not handling hostnames without dot characters [release] (Jeffrey Ryan)
| * DNSResolver: fix not handling hostnames without dot characters [release]Jeffrey Ryan2022-11-211-0/+11
| | | | | | | | Unrelated, but similar code-wise to #8643. There is a check in `DNSResolver` which automatically fails to resolve hostnames which do not contain the `.` character. This PR removes that check.
* | Merge pull request #8644luigi11112023-01-111-0/+35
|\ \ | | | | | | | | | 38d4811 p2p: fix exclusive node DNS resolution for certain hosts [release] (Jeffrey Ryan)
| * | p2p: fix exclusive node DNS resolution for certain hosts [release]Jeffrey Ryan2022-11-171-0/+35
| |/ | | | | | | | | | | | | | | Fixes #8633. The function `append_net_address` did not parse hostname + port addresses (e.g. `bar:29080`) correctly if the hostname did not contain a `'.'` character. @vtnerd comments 1 clear up 2nd conditional statement
* / Fix dandelion++ fluff/stem bug with local txesLee Clagett2022-10-251-15/+84
|/
* Merge pull request #8588luigi11112022-09-262-0/+2
|\ | | | | | | 802c4bb Move update_checkpoints() to a later stage (SChernykh)
| * Move update_checkpoints() to a later stageSChernykh2022-09-222-0/+2
| | | | | | | | update_checkpoints() makes a few DNS requests and can take up to 20-30 seconds to complete (3-6 seconds on average). It is currently called from core::handle_incoming_block() which holds m_incoming_tx_lock, so it blocks all incoming transactions and blocks processing while update_checkpoints() is running. This PR moves it to until after a new block has been processed and relayed, to avoid full monerod locking.
* | Merge pull request #8582luigi11112022-09-261-24/+64
|\ \ | | | | | | | | | 1cd21bf add an option to force-update multisig key exchange under some circumstances (koe)
| * | add an option to force-update multisig key exchange under some circumstanceskoe2022-09-211-24/+64
| |/
* / Second thread pool for IOSChernykh2022-09-201-1/+1
|/
* Merge pull request #8545v0.18.1.1luigi11112022-09-151-1/+1
|\ | | | | | | | | | | | | 12e7c41 Merge pull request #5 from j-berman/restore-msig-encrypted-seed (Justin Berman) 848a0c0 Fix segfault restoring encrypted multisig seed (j-berman) 401f5d9 Require user ack multisig is experimental to restore (j-berman) fc8a5d6 multisig: fix #8537 seed restore (suggestions by @UkoeHB) (j-berman)
| * multisig: fix #8537 seed restore (suggestions by @UkoeHB)j-berman2022-09-011-1/+1
| | | | | | | | | | | | | | | | - spend secret key is no longer the sum of multisig key shares; no need to check that is the case upon restore. - restoring a multisig wallet from multisig info means that the wallet must have already completed all setup rounds. Upon restore, set the number of rounds completed accordingly.
* | wallet2: check wallet compatibility with daemon's hard fork versionj-berman2022-09-121-1/+1
| |
* | wallet2: better test on whether to allow output importmoneromooo-monero2022-09-071-0/+9
| | | | | | | | | | | | | | | | Being offline is not a good enough heuristic, so we keep track of whether the wallet ever refreshed from a daemon, which is a lot better, and probably the best we can do without manual user designation (which would break existing cold wallet setups till the user designates those wallets)
* | allow exporting outputs in chunksmoneromooo-monero2022-09-073-15/+41
| | | | | | | | | | this will make it easier huge wallets to do so without hitting random limits (eg, max string size in node).
* | wallet2: fixes for export/import output flowj-berman2022-09-062-18/+67
|/ | | | | | - only allow offline wallets to import outputs - don't import empty outputs - export subaddress indexes when exporting outputs
* Merge pull request #8483luigi11112022-08-073-17/+96
|\ | | | | | | 6075be9 feat(trezor): add HF15 support, BP+ (Dusan Klinec)
| * feat(trezor): add HF15 support, BP+Dusan Klinec2022-08-053-17/+96
| | | | | | | | | | - BP+ support added for Trezor - old Trezor firmware version support removed, code cleanup
* | Publish submitted txs via zmqj-berman2022-07-214-5/+25
|/
* Merge pull request #8426luigi11112022-07-151-2/+252
|\ | | | | | | | | | | a82fba4 address PR comments (j-berman) 3be1dbd connection: fix implementation (anon) 724ff21 connection: add segfault and deadlocks demo (anon)
| * address PR commentsj-berman2022-07-081-5/+5
| |
| * connection: add segfault and deadlocks demoanon2022-07-051-2/+252
| |
* | derive multisig tx secret keys from an entropy source plus the tx inputs' ↵koe2022-07-131-2/+3
| | | | | | | | key images
* | multisig: fix critical vulnerabilities in signinganon2022-06-3011-124/+114
|/
* Merge pull request #8340luigi11112022-05-265-0/+42
|\ | | | | | | | | 08080df unit_tests: add more sha256sum test cases (Jeffrey Ryan) a66a52d common: update sha256sum to use OpenSSL 3.0 API (Jeffrey Ryan)