aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Makefile: fix spelling of CMAKE_BUILD_TYPE valueselsta2022-04-082-13/+13
|
* Merge pull request #8242luigi11112022-04-061-3/+1
|\ | | | | | | 6abe3fe tests: integrated_address: remove missing payment id assertion (reemuru)
| * tests: integrated_address: remove missing payment id assertionreemuru2022-04-061-3/+1
|/ | | | | | | | The integrated address functional test fails in the workflows due to an assertion for missing payment id that is no longer needed. Remove the assertion and update the assertion count. Fixes 7dcfccb: ("wallet_rpc_server: fix make_integrated_address with no payment id")
* Merge pull request #8228luigi11112022-04-063-141/+72
|\ | | | | | | 1ce9e9c Remove dead code from parserse_base_utils and fix unit tests (Jeffrey)
| * Remove dead code from parserse_base_utils and fix unit testsJeffrey2022-03-303-141/+72
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Remove `match_string()`, `match_number()`, and `match_word()` * Remove `match_word_with_extrasymb()` and `match_word_til_equal_mark()` * Adapt unit test for `match_number()` to `match_number2()` * Adapt unit test for `match_string()` to `match_string2()` Note: the unit tests were testing for the old version of the functions, and the interfaces for these functions changed slightly, so I had to also edit the tests. As of writing, this PR has no merge conflicts with #8211 Additional changes during review: * Explicitly set up is_[float/signed]_val to be changed before each call * Structify the tests and fix uninitialized variables
* | Merge pull request #8223luigi11112022-04-0610-50/+43
|\ \ | | | | | | | | | 17772ef Eliminate dependence on boost::interprocess #8223 (Jeffrey)
| * | Eliminate dependence on boost::interprocess #8223Jeffrey2022-03-3010-50/+43
| |/ | | | | | | | | | | | | | | | | | | | | | | | | In this repo, `boost::interprocess` was being used soley to make `uint32_t` operations atomic. So I replaced each instance of `boost::interprocess::ipcdetail::atomic(...)32` with `std::atomic` methods. I replaced member declarations as applicable. For example, when I needed to change a `volatile uint32_t` into a `std::atomic<uint32_t>`. Sometimes, a member was being used a boolean flag, so I replaced it with `std::atomic<bool>`. You may notice that I didn't touch `levin_client_async.h`. That is because this file is entirely unused and will be deleted in PR monero-project#8211. Additional changes from review: * Make some local variables const * Change postfix operators to prefix operators where value was not need
* | Merge pull request #8215luigi11112022-04-062-3/+13
|\ \ | | | | | | | | | d1f1947 wallet_rpc_server: support regex for get_accounts tag (reemuru)
| * | wallet_rpc_server: support regex for get_accounts tagreemuru2022-03-132-3/+13
| | | | | | | | | | | | | | | | | | | | | | | | This commit adds a 'regexp' boolean field to the get_accounts request. The flag is set to false by default and maintains backwards compatibility. When set to true the user can search tags by regular expression filters. An additional error message was added for failed regular expression searches. Bump minor version to 25.
* | | Merge pull request #8213luigi11112022-04-061-6/+0
|\ \ \ | | | | | | | | | | | | 7dcfccb wallet_rpc_server: fix make_integrated_address with no payment id (moneromooo-monero)
| * | | wallet_rpc_server: fix make_integrated_address with no payment idmoneromooo-monero2022-03-111-6/+0
| |/ /
* | | Merge pull request #8212luigi11112022-04-061-3/+0
|\ \ \ | | | | | | | | | | | | 591c174 Remove footgun doc comment in miner TX validation (Luke Parker)
| * | | Remove footgun doc comment in miner TX validationLuke Parker2022-03-101-3/+0
| |/ / | | | | | | This comment suggests this check is unnecessary, when it is completely necessary as miner TXs can have multiple outputs *which is a statement directly, and incorrectly, contradicted by this comment*. While I don't ever see someone removing this code and getting their edits merged into Monero, someone inexperienced who thinks they're cleaning old code may break their own work, and then there's really just zero benefit to keeping this around.
* | | Merge pull request #8205luigi11112022-04-062-1/+2
|\ \ \ | | | | | | | | | | | | 0d6fce8 Compil. time: cryptonote_core minus portable_storage header (mj-xmr)
| * | | Compil. time: cryptonote_core minus portable_storage headermj-xmr2022-03-042-1/+2
| |/ /
* | | Merge pull request #8197luigi11112022-04-06695-695/+793
|\ \ \ | | | | | | | | | | | | da9aa1f Copyright: Update to 2022 (mj-xmr)
| * | | Copyright: Update to 2022mj-xmr2022-03-04695-695/+793
| |/ /
* | | Merge pull request #8195luigi11112022-04-061-4/+4
|\ \ \ | | | | | | | | | | | | 3c82958 wallet2: update stagenet rollback blocks (selsta)
| * | | wallet2: update stagenet rollback blocksselsta2022-03-011-4/+4
| | | |
* | | | Merge pull request #8180luigi11112022-04-061-13/+47
|\ \ \ \ | | | | | | | | | | | | | | | 3f88fdd simplewallet: add option for exporting tx keys (reemuru)
| * | | | simplewallet: add option for exporting tx keysreemuru2022-02-241-13/+47
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | This commit adds an option to export transaction keys when performing export_transfers all. By passing option=with_keys a new column 'tx key' will be populated with respective keys.
* | | | Merge pull request #8179luigi11112022-04-062-10/+167
|\ \ \ \ | | | | | | | | | | | | | | | ae0a840 wallet2: decrease the amount of data exchanged for output export (moneromooo-monero)
| * | | | wallet2: decrease the amount of data exchanged for output exportmoneromooo-monero2022-03-292-10/+167
| | |/ / | |/| |
* | | | Merge pull request #8053luigi11112022-04-061-2/+2
|\ \ \ \ | | | | | | | | | | | | | | | cf44349 account info text alignment (im)
| * | | | account info text alignmentim2021-11-101-2/+2
| | | | |
* | | | | Merge pull request #7937luigi11112022-04-061-14/+36
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | 83bb027 Make RPC server functions that read db thread safe (j-berman)
| * | | | | Make RPC server functions that read db thread safej-berman2022-04-051-14/+36
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - grab an lmdb db_rtxn_guard to ensure consistent data from the db - fixed on_getblockhash error resp when requested height >= blockchain height - left functions that read shared memory untouched for now
* | | | | | Merge pull request #7170luigi11112022-04-0537-165/+2674
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | bd27deb Bulletproofs+ (SarangNoether) b7713cc Precompute initial transcript hash (SarangNoether) b535d66 Updates from security audit (SarangNoether) a0d80b1 plug bulletproofs plus into consensus (moneromooo-monero) 75bd004 ringct: a few minor optimizations from review (moneromooo-monero) a345060 ringct: port some of vtnerd's review changes from BP+ to BP (moneromooo-monero) 4c94cfe store outPk/8 in the tx for speed (moneromooo-monero) 5acdd0e bulletproofs+: some minor cleanup from vtnerd's review (moneromooo-monero)
| * | | | | | bulletproofs+: some minor cleanup from vtnerd's reviewmoneromooo-monero2022-04-051-11/+2
| | | | | | |
| * | | | | | store outPk/8 in the tx for speedmoneromooo-monero2022-04-055-7/+31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It avoids dividing by 8 when deserializing a tx, which is a slow operation, and multiplies by 8 when verifying and extracing the amount, which is much faster as well as less frequent
| * | | | | | ringct: port some of vtnerd's review changes from BP+ to BPmoneromooo-monero2022-04-051-13/+11
| | | | | | |
| * | | | | | ringct: a few minor optimizations from reviewmoneromooo-monero2022-04-051-18/+15
| | | | | | |
| * | | | | | plug bulletproofs plus into consensusmoneromooo-monero2022-04-0528-159/+1137
| | | | | | |
| * | | | | | Updates from security auditSarang Noether2022-04-041-2/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | https://suyash67.github.io/homepage/assets/pdfs/bulletproofs_plus_audit_report_v1.1.pdf
| * | | | | | Precompute initial transcript hashSarang Noether2022-04-041-13/+11
| | | | | | |
| * | | | | | Bulletproofs+Sarang Noether2022-04-0411-2/+1517
| | |_|_|_|/ | |/| | | |
* | | | | | Merge pull request #7153luigi11112022-04-054-0/+38
|\ \ \ \ \ \ | |/ / / / / |/| | | | | | | | | | | 354e1d9 Make the wallet name optional when locked. (Norman Moeschter)
| * | | | | Make the wallet name optional when locked.Norman Moeschter2022-03-104-0/+38
| | |_|/ / | |/| | |
* | | | | Merge pull request #8216luigi11112022-03-181-1/+1
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | f22051d replace erciccione's seednode with one on haveno's infrastructure (erciccione)
| * | | | | replace erciccione's seednode with one on haveno's infrastructureerciccione2022-03-131-1/+1
| |/ / / /
* | | | | Merge pull request #8187luigi11112022-03-181-6/+0
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | 9bde2db Fixes: Don't require --rpc-login with --rpc-access-control-origins #8168 (Jeffrey)
| * | | | | Fixes: Don't require --rpc-login with --rpc-access-control-origins #8168Jeffrey2022-02-191-6/+0
| | |_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This PR removes the requirement for --rpc-login to be specified if --rpc-access-control-origins is. This will allow public nodes to serve cross-origin requests. You can still use --rpc-login with --rpc-access-control-origins, but it is no longer mandatory. Original Issue: #8168
* | | | | Merge pull request #8186luigi11112022-03-186-12/+24
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | f68f927 Undefined behaviour fixes (SChernykh)
| * | | | | Undefined behaviour fixesSChernykh2022-02-196-12/+24
| |/ / / / | | | | | | | | | | | | | | | Fixes issues reported in #8120
* | | | | Merge pull request #8170luigi11112022-03-181-17/+0
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | 14eb42d systemd: remove outdated torsocks example (selsta)
| * | | | | systemd: remove outdated torsocks exampleselsta2022-02-081-17/+0
| |/ / / /
* | | | | Merge pull request #8169luigi11112022-03-182-1/+4
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | 5e201fe rpc: add explicit restricted flag to /get_info (tobtoht)
| * | | | | rpc: add explicit restricted flag to /get_infotobtoht2022-02-072-1/+4
| |/ / / /
* | | | | Merge pull request #8158luigi11112022-03-182-5/+15
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | e5000a9 Balance includes unconfirmed transfers to self (woodser)
| * | | | | Balance includes unconfirmed transfers to selfwoodser2022-01-312-5/+15
| |/ / / /