aboutsummaryrefslogtreecommitdiff
path: root/tests/unit_tests
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | p2p: allow comments in banlist filesjeffro2562024-12-131-0/+85
| |/ / | | | | | | | | | In-line comments explicitly explaining banned hosts/subnets might help assuage fears of some good banlists' arbitaryiness.
* / / contrib: force (de)serialization to create params section incase there is none.0xFFFC00002024-12-131-0/+68
|/ / | | | | | | Co-authored-by: Boog900 <boog900@tutanota.com>
* | build: fix build with Boost 1.85 and remove instances of viewkey logging ↵jeffro2562024-09-103-3/+3
| | | | | | | | | | | | | | | | | | | | | | [RELEASE] 1. Use std::is_standard_layout and std::is_trivially_copyable instead of std::is_pod for KV byte-wise serialization, which fixes compile issue for Boost UUIDs 2. Removed reimplementation of std::hash for boost::uuids::uuid 3. Removed << operator overload for crypto::secret_key 4. Removed instances in code where private view key was dumped to the log in plaintext Release version of #9450, containing C++14 modified assertions
* | epee: string_tools: keep full path in cut_off_extensiontobtoht2024-08-141-0/+7
| |
* | epee: string_tools: remove dot from get_extensiontobtoht2024-08-141-0/+8
| | | | | | | | | | Fixes a regression introduced in #9254. Previously it did not include the dot.
* | Merge pull request #9385luigi11112024-08-131-0/+62
|\ \ | | | | | | | | | c5ad937 Fix ZMQ DaemonInfo: (Lee *!* Clagett)
| * | Fix ZMQ DaemonInfo:Lee *!* Clagett2024-06-241-0/+62
| |/ | | | | | | | | | | * top_block_hash was never set in handler * wide_difficulty was never sent in JSON * wide_cumulative_difficulty was never sent in JSON
* / cryptonote_protocol: prevent duplicate txs in fluff queue0xFFFC00002024-06-061-0/+57
|/ | | | | | | 1. Fix duplicate transaction #9335 2. Add test for cases where there are duplicate transaction in fluff Co-authored-by: Boog900 <boog900@tutanota.com>
* Merge pull request #9311luigi11112024-05-201-1/+1
|\ | | | | | | dd47d03 Enforce Tx unlock_time is Zero by Relay Rule [RELEASE] (jeffro256)
| * Enforce Tx unlock_time is Zero by Relay Rule [RELEASE]jeffro2562024-04-291-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Related to https://github.com/monero-project/research-lab/issues/78 Added a relay rule that enforces the `unlock_time` field is equal to 0 for non-coinbase transactions. UIs changed: * Removed `locked_transfer` and `locked_sweep_all` commands from `monero-wallet-cli` APIs changed: * Removed `unlock_time` parameters from `wallet2` transfer methods * Wallet RPC transfer endpoints send error codes when requested unlock time is not 0 * Removed `unlock_time` parameters from `construct_tx*` cryptonote core functions @tobtoht: undo rebase changes tx.dsts -> tx_dsts
* | Merge pull request #9306luigi11112024-05-201-6/+7
|\ \ | | | | | | | | | 755dddd common: support boost filesystem copy_options. Co-authored-by: selsta <selsta@sent.at> (0xFFFC0000)
| * | common: support boost filesystem copy_options.0xFFFC00002024-05-011-6/+7
| |/ | | | | | | Co-authored-by: selsta <selsta@sent.at>
* / Skip privacy networks (on tx sends) that don't have outgoing connectionsLee *!* Clagett2024-03-291-0/+33
|/
* 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-201-0/+57
|/ | | | | | | | | | | | | | | | | 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
| |
* | wallet2: fix `store_to()` and `change_password()`jeffro2562023-08-232-0/+267
|/ | | | | | | | 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
* common: do not use DNS to determine if address is localtobtoht2023-05-252-0/+51
| | | | Co-authored-by: j-berman <justinberman@protonmail.com>
* verRctNonSemanticsSimpleCached: fix fragilityJeffrey Ryan2023-03-172-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-261-0/+1
|\ | | | | | | 802c4bb Move update_checkpoints() to a later stage (SChernykh)
| * Move update_checkpoints() to a later stageSChernykh2022-09-221-0/+1
| | | | | | | | 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.
* | add an option to force-update multisig key exchange under some circumstanceskoe2022-09-211-24/+64
|/
* 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.
* 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
| |
* | multisig: fix critical vulnerabilities in signinganon2022-06-303-20/+21
|/
* Merge pull request #8340luigi11112022-05-261-0/+22
|\ | | | | | | | | 08080df unit_tests: add more sha256sum test cases (Jeffrey Ryan) a66a52d common: update sha256sum to use OpenSSL 3.0 API (Jeffrey Ryan)
| * unit_tests: add more sha256sum test casesJeffrey Ryan2022-05-181-0/+22
| |
* | Don't exclusively drop tor/i2p outgoing cxns in idle loopj-berman2022-05-181-2/+2
|/
* Merge pull request #8302luigi11112022-05-102-2/+2
|\ | | | | | | 41da2fe Update copyright to 2022 for Hardfork files (Akrit)
| * Update copyright to 2022 for Hardfork filesAkrit2022-04-292-2/+2
| | | | | | | | Update Makefile and LICENSE
* | Merge pull request #8220luigi11112022-05-101-8/+7
|\ \ | |/ |/| | | 0d6ecb1 multisig: add post-kex verification round to check that all participants have completed the multisig address (koe)
| * multisig: add post-kex verification round to check that all participants ↵koe2022-04-291-8/+7
| | | | | | | | have completed the multisig address
* | Add Include statementsJeffrey2022-04-181-0/+1
| |
* | Fee changes from ArticMinemoneromooo-monero2022-04-102-0/+188
| | | | | | | | | | | | https://github.com/ArticMine/Monero-Documents/blob/master/MoneroScaling2021-02.pdf with a change to use 1.7 instead of 2.0 for the max long term increase rate
* | epee: allow copying a rolling_median_t objectmoneromooo-monero2022-04-101-0/+18
| |
generated by cgit v1.2.3 (git 2.51.0) at 2026-08-16 06:20:19 +0000